I was having weird issues with running Golang CI locally, and had to match the Go version.

Tests generated by gemini CLI

I wish I knew more about Coder templates. Do templates have a source field (or anything equivalent) that might need to be validated, too?

no its not applicable to templates

I think this can be rewritten as:

if strings.HasPrefix(nextLine, "```") {
	if strings.HasPrefix(nextLine, "```tf") {
		isInsideTerraform = true
		continue
	}
	if isInsideTerraform {
		break
	}
}

I don't see the point of the firstTerraformBlock variable. If we always break at the end of the first Terraform code block, we don't really need to track anything extra

The goal is to check all tf blocks.

Maybe I'm missing something obvious, then. What validation requirement(s) are we trying to fulfill?

I was under the impression that we wanted the first Terraform block to contain a source field, but that other sources could be valid for the other blocks. Are we trying to keep all sources fully "boxed in" for Coder templates?

Not saying that the current code is wrong, but I'm not fully sure what we're trying to protect against, so I just want to make sure I'm giving good feedback

We want to make sure all sources for all codeblock in a module README.md are in correct format. This should not apply to templates if it does its probably a bug.

Sorry, I meant "modules" instead of "templates"

But if the goal is to check all Terraform blocks, then we'll need to remove the firstTerraformBlock variable completely.

With how the code is currently set up, once we hit the end of the first block, we always hit the else if block and immediately break out of the loop. We would need to update the loop logic to keep going after reaching the end of a block – flipping isInsideTerraform back to false and then probably just continue