AI Slop Code Risk Monitor

hashicorp/terraform

SlopCodeMonitor badge for hashicorp/terraform Adjusted legacy signal; badge bands: 25 / 50 / 75 (different from dashboard bands).

Terraform enables you to safely and predictably create, change, and improve infrastructure. It is a source-available tool that codifies APIs into declarative configuration files that can be shared amongst team members, treated as code, edited, reviewed, and versioned.

2.4 Low pattern signal View on GitHub

Open-Source Code Quality, Security and Technical Debt Review

This report summarizes scanner evidence for hashicorp/terraform, a Go project with 50,119 GitHub stars. SynthScan examined 778,323 lines of code across 4941 source files, recording 1498 pattern matches distributed across 13 syntactic categories. The legacy adjusted pattern signal is 2.4 (Low pattern signal); it is not an authorship estimate.

Use the findings to ask whether maintainers can explain intent, verify dependencies, simplify unnecessary scaffolding, and focus review effort. Risk Payload v1 adds ownership, supply-chain, over-engineering, and review-effort vectors when available. These are prompts for human verification, not probabilities, security findings, or measured maintenance costs.

2.4
Adjusted Signal
2.4
Raw Pattern Score
100%
Time Factor
2026-10-30
Last Push
50.1K
Stars
Go
Language
778.3K
Lines of Code
4.9K
Files
1.5K
Pattern Hits
2026-10-31
Scan Date
0.00
HC Hit Rate

Risk Review · Payload v1

Use each vector to guide manual verification of the underlying findings.

Payload schema: 1.0.0 · Engine: 2.0.0 · Scoring: heuristic-density-v1 · Scan date: 2026-10-31.

Cognitive Ownership Risk 10.2/100 · LOW

Can maintainers explain intent, assumptions, and trade-offs? 1419 finding(s).

Hallucinated Dependency & Supply Chain Risk 1.2/100 · LOW

Are imports declared and package/API names verified? 79 finding(s).

Agent Over-Engineering Ratio 0.0/100 · LOW

Are wrappers, guards, and repeated structures necessary? 2 finding(s).

PR Review Velocity Impact 11.3/100 · LOW

Which changes need focused human review? 1498 finding(s).

What These Metrics Mean

Adjusted Signal
Legacy pattern score normalized per 1,000 scanned lines and multiplied by the repository-level time factor. It is not an estimate of AI authorship.
Raw Pattern Score
Sum of severity- and context-weighted pattern findings before applying the time factor.
Time Factor
A coarse multiplier based on repository last-push metadata. It does not identify when individual code was written.
Pattern Hits
Count of configured pattern findings. Review each location and surrounding code before acting.
HC Hit Rate
High- and critical-severity findings per scanned file. Severity is a configured detector weight, not proof of a vulnerability or authorship.
Lines / Files
Scan coverage denominators used by the legacy normalized score.

Pattern-Signal History

This chart tracks the legacy adjusted pattern score across scan runs. Changes can reflect source changes, scan coverage, or detector/scoring versions; they do not measure AI adoption or prove that a team refactored code. Compare runs only when their scoring versions are compatible. The dashed line tracks the raw pattern-hit count.

Severity Breakdown

Groups findings by configured severity weight. Severity is not a confidence estimate, security rating, or proof that a package is missing. Verify imports against the project's manifests and inspect each finding in its local design context.

CRITICAL 12HIGH 5MEDIUM 117LOW 1364

Directory Score Breakdown

This horizontal bar chart shows where configured pattern scores accumulate across directories. Treat the bars as locations to inspect, not as evidence of authorship or code quality. Use the file-level findings to decide whether a directory needs a focused ownership, dependency, complexity, or review check.

Pattern Findings

The scanner identified 1498 distinct pattern matches across 13 syntactic categories. Each entry below represents a discrete location in the source code where the engine recorded a detector match to inspect in context. Expand any category row to inspect the individual file paths, line numbers, code snippets, and the lexical context (CODE, COMMENT, or STRING) in which each match was detected.

Reading the findings table: The Severity column indicates the configured finding weight (CRITICAL / HIGH / MEDIUM / LOW). The Context column identifies whether the match occurred inside executable code, an inline comment, or a string literal — legacy comment-context matches receive a ×1.5 score weight. The ⚡ bolt icon marks clustered matches: three or more patterns within a 10-line window, each receiving an additional ×1.5 density multiplier under the legacy scoring model.

Over-Commented Block1269 hits · 1269 pts
SeverityFileLineSnippetContext
LOWtelemetry.go21// If this environment variable is set to "otlp" when running Terraform CLICOMMENT
LOWtelemetry.go41// By default we don't export telemetry information at all, since Terraform isCOMMENT
LOWprovider_source.go81 return getproviders.MultiSource(searchRules), diagsCOMMENT
LOWprovider_source.go101 // following e.g. the XDG base directory specification on Unix systems,COMMENT
LOWexperiments.go1// Copyright IBM Corp. 2014, 2026COMMENT
LOWmain.go301 // actually known. If not, we might be able to suggest an alternativeCOMMENT
LOWtools/loggraphdiff/loggraphdiff.go1// Copyright IBM Corp. 2014, 2026COMMENT
LOWtools/protobuf-compile/protobuf-compile.go1// Copyright IBM Corp. 2014, 2026COMMENT
LOWinternal/terraform/context_apply_test.go2281 if actual != expected {COMMENT
LOWinternal/terraform/context_apply_test.go5641 // if new, ok := rc.Get("foo"); ok {COMMENT
LOWinternal/terraform/context_apply_test.go7241 if len(mod.Resources) != 0 {COMMENT
LOWinternal/terraform/context_import.go41 // LegacyID stores the ID from the command line arguments when using theCOMMENT
LOWinternal/terraform/transform_reference.go21//COMMENT
LOWinternal/terraform/transform_reference.go81// not yet expended in the graph. While this will cause some extra dataCOMMENT
LOWinternal/terraform/transform_reference.go101// relative to the returned selfPath.COMMENT
LOWinternal/terraform/config_graph_build_test.go261COMMENT
LOWinternal/terraform/context_apply.go21 "github.com/hashicorp/terraform/internal/tfdiags"COMMENT
LOWinternal/terraform/context_apply.go41 // values that were declared as ephemeral, because all other inputCOMMENT
LOWinternal/terraform/context_apply.go61COMMENT
LOWinternal/terraform/context_apply.go81 AllowRootDeprecatedOutputs: po.AllowRootDeprecatedOutputs,COMMENT
LOWinternal/terraform/node_resource_plan_instance.go121 diags = diags.Append(planDiags)COMMENT
LOWinternal/terraform/node_resource_plan_instance.go621 // replacement, but we still overload forceReplace to get the correctCOMMENT
LOWinternal/terraform/node_resource_plan_instance.go801 if n.Config == nil {COMMENT
LOWinternal/terraform/evaluate_placeholder.go21// only what we know to be true for all possible final module instancesCOMMENT
LOWinternal/terraform/evaluate_placeholder.go41 Operation walkOperationCOMMENT
LOWinternal/terraform/evaluate_placeholder.go181COMMENT
LOWinternal/terraform/node_module_variable.go121 // Otherwise, we depend on anything referenced by our value expression.COMMENT
LOWinternal/terraform/node_module_variable.go381 // module instances under our partially-expanded prefix, and to recordCOMMENT
LOWinternal/terraform/context_plan2_test.go861`})COMMENT
LOWinternal/terraform/context_plan2_test.go8121COMMENT
LOWinternal/terraform/transform_variable_validation.go21 dag.VertexCOMMENT
LOWinternal/terraform/transform_variable_validation.go41}COMMENT
LOWinternal/terraform/transform_targets.go61 // this node. In case of resource nodes, action triggers are considered closely relatedCOMMENT
LOWinternal/terraform/context_eval.go21 // ExternalProviders is a set of pre-configured provider instances withCOMMENT
LOWinternal/terraform/context_eval.go121// evalScopeFromGraphWalk takes a [ContextGraphWalker] that was already usedCOMMENT
LOWinternal/terraform/node_resource_abstract_instance.go941 // we're creating an entirely new object, but then turn it intoCOMMENT
LOWinternal/terraform/node_resource_abstract_instance.go1201 }COMMENT
LOWinternal/terraform/node_resource_abstract_instance.go1841 }COMMENT
LOWinternal/terraform/node_resource_abstract_instance.go1861// of finishing during the plan are:COMMENT
LOWinternal/terraform/node_resource_abstract_instance.go2041 // If we are a nested block, then we want to create a plannedChange thatCOMMENT
LOWinternal/terraform/node_resource_abstract_instance.go2821 }COMMENT
LOWinternal/terraform/node_resource_abstract_instance.go2861 }COMMENT
LOWinternal/terraform/node_resource_abstract_instance.go2881 // root cause rather than distract with this extra problem.COMMENT
LOWinternal/terraform/eval_variable.go141 Summary: "Invalid value for input variable",COMMENT
LOWinternal/terraform/eval_variable.go281 //COMMENT
LOWinternal/terraform/eval_variable.go341 // Terraform would treat error messages specified using JSONCOMMENT
LOWinternal/terraform/context_apply2_test.go6441// The configuration takes precedence over a stored dependency which conflictsCOMMENT
LOWinternal/terraform/node_variable_validation.go21// A node of this type should always depend on another node that's responsibleCOMMENT
LOWinternal/terraform/node_variable_validation.go61// this node contributes to the value for the input variable that it'sCOMMENT
LOWinternal/terraform/context_input.go21// configurations.COMMENT
LOWinternal/terraform/context_apply_policy_test.go2061// TestContext2Apply_PolicyPhaseSpanOutlivesEvaluations verifies that theCOMMENT
LOWinternal/terraform/transform_destroy_edge_test.go221 }COMMENT
LOWinternal/terraform/transform_destroy_edge_test.go301}COMMENT
LOWinternal/terraform/transform_config.go21// action declarations from the configuration to the graph.COMMENT
LOWinternal/terraform/transform_module_policy.go81 return nilCOMMENT
LOWinternal/terraform/transform_module_policy.go101 // input2 = module.child.outputCOMMENT
LOWinternal/terraform/config_graph_module_test.go421// implied based on the resource type. For example, this resource:COMMENT
LOWinternal/terraform/validate_selfref.go61// for_each) does not reference the resource it is attached to. The behaviourCOMMENT
LOWinternal/terraform/hook.go61// parts of Terraform, allowing you to inspect or change behavior at runtime.COMMENT
LOWinternal/terraform/hook.go81 // provisioning for a particular instance and of each of the individualCOMMENT
1209 more matches not shown…
Self-Referential Comments76 hits · 205 pts
SeverityFileLineSnippetContext
MEDIUM…urcebundle/foo/non-empty-stack/empty-module/nothing.tf1# This module is intentionally empty, since it's called from stackCOMMENT
MEDIUMinternal/depsfile/locks_file.go135 Bytes: []byte("# This file is maintained automatically by \"terraform init\".\n"),CODE
MEDIUMinternal/depsfile/locks_file_test.go248 wantContent := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/providers_lock_test.go45 expected := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/providers_lock_test.go61 expected := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/providers_lock_test.go78 expected := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/init_test.go2784 expectedContent := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/init_test.go3384# This file is maintained automatically by "terraform init".COMMENT
MEDIUMinternal/command/init_test.go3411# This file is maintained automatically by "terraform init".COMMENT
MEDIUMinternal/command/init_test.go3424# This file is maintained automatically by "terraform init".COMMENT
MEDIUMinternal/command/init_test.go3437# This file is maintained automatically by "terraform init".COMMENT
MEDIUMinternal/command/init_test.go3451# This file is maintained automatically by "terraform init".COMMENT
MEDIUMinternal/command/state_migrate_test.go542 expectedContents := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/state_migrate_test.go568 lockFileContents := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/state_migrate_test.go793 expectedContents := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/state_migrate_test.go818 lockFileContents := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/state_migrate_test.go945 lockFileContents := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/state_migrate_test.go1168 lockFileContents := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/state_migrate_test.go1299 lockFileContents := `# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/state_migrate_test.go1322 lockFileContents := `# This file is maintained automatically by "terraform init".CODE
MEDIUM…migrate-state-store-to-state-store/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…re-unchanged/dev-override-provider/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…nged/provider-managed-by-terraform/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…mand/testdata/plan-out-state-store/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…tdata/provider-schemas-state-store/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…ate-migrate-state-store-to-backend/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…tate-migrate-missing-migrate-files/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…ase/config-state-file-and-lockfile/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…oad-prerelease/state-and-lock-file/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…a/init-get-providers-with-lockfile/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…a/state-migrate-backend-to-backend/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…oad/config-state-file-and-lockfile/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…e-changed/provider-version-changed/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…/state-store-changed/provider-used/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…tate-store-changed/provider-config/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…ate-migrate-backend-to-state-store/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…stdata/state-push-state-store-good/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…data/state-migrate-empty-mod-files/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…and/testdata/providers-lock/append/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…ta/state-migrate-missing-mod-files/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUMinternal/command/e2etest/init_test.go485 expectedLockFileContent := fmt.Sprintf(`# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/e2etest/provider_plugin_test.go242 expectedLockFileContent := fmt.Sprintf(`# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/e2etest/provider_plugin_test.go263 priorLockFile := fmt.Sprintf(`# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/e2etest/provider_plugin_test.go319 priorLockFile := fmt.Sprintf(`# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/e2etest/provider_plugin_test.go369 expectedLockFileContent := fmt.Sprintf(`# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/e2etest/provider_plugin_test.go513 expectedLockFileContent := fmt.Sprintf(`# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/e2etest/provider_plugin_test.go533 priorLockFile := fmt.Sprintf(`# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/e2etest/provider_plugin_test.go588 priorLockFile := fmt.Sprintf(`# This file is maintained automatically by "terraform init".CODE
MEDIUMinternal/command/e2etest/provider_plugin_test.go637 expectedLockFileContent := fmt.Sprintf(`# This file is maintained automatically by "terraform init".CODE
MEDIUM…t/testdata/state-migrate-upgrade-2/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…rectory-with-state-store-unmanaged/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…iders-mirror-with-broken-lock-file/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…rm-providers-mirror-with-lock-file/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…est/testdata/state-migrate-upgrade/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…ized-directory-with-state-store-fs/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…ests/tests/moved_with_refresh_only/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…sts/tests/basic_json_string_update/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…ence-tests/tests/moved_with_update/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…nce-tests/tests/replace_within_set/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
MEDIUM…uivalence-tests/tests/drift_simple/.terraform.lock.hcl1# This file is maintained automatically by "terraform init".COMMENT
16 more matches not shown…
Hallucination Indicators12 hits · 120 pts
SeverityFileLineSnippetContext
CRITICAL…time/internal/stackeval/removed_stack_call_instance.go69 validateStackCallInstanceInputsFn(r.Stack(ctx, phase), r.call.config.config.Inputs, r.call.config.config.DeclRange.ToHCODE
CRITICAL…/stackruntime/internal/stackeval/component_instance.go715 return c.call.config.config.DeclRange.ToHCL().Ptr()CODE
CRITICAL…ternal/stacks/stackruntime/internal/stackeval/stack.go643 Detail: fmt.Sprintf("The `from` attribute resolved to component instance %s, which is already claimed by anotherCODE
CRITICAL…ternal/stacks/stackruntime/internal/stackeval/stack.go644 Subject: inst.call.config.config.DeclRange.ToHCL().Ptr(),CODE
CRITICAL…ternal/stacks/stackruntime/internal/stackeval/stack.go666 Detail: fmt.Sprintf("The `from` attribute resolved to stack instance %s, which is already claimed by another remCODE
CRITICAL…ternal/stacks/stackruntime/internal/stackeval/stack.go667 Subject: inst.call.config.config.DeclRange.ToHCL().Ptr(),CODE
CRITICAL…ntime/internal/stackeval/removed_component_instance.go332 return r.call.config.config.DeclRange.ToHCL().Ptr()CODE
CRITICAL…cks/stackruntime/internal/stackeval/provider_config.go134 versionConstraints, _ := p.stack.config.Stack.RequiredProviders.VersionConstraintsForProvider(providerType.Addr())CODE
CRITICAL…stackruntime/internal/stackeval/stack_call_instance.go117 validateStackCallInstanceInputsFn(c.Stack(ctx, phase), c.call.config.config.Inputs, c.call.config.config.DeclRange.ToHCODE
CRITICAL…/stackruntime/internal/stackeval/removed_stack_call.go141 Subject: rsc.call.config.config.DeclRange.ToHCL().Ptr(),CODE
CRITICALinternal/command/jsonplan/values.go193 r.Addr.Resource.Resource.Mode.String(),CODE
CRITICALinternal/moduletest/graph/node_test_run_cleanup.go93 Subject: n.run.Config.Backend.DeclRange.Ptr(),CODE
AI Slop Vocabulary38 hits · 110 pts
SeverityFileLineSnippetContext
MEDIUMinternal/terraform/eval_variable.go38 // but we'll check it here too to be robust.COMMENT
MEDIUMinternal/terraform/eval_variable_test.go104 // metadata is stripped from empty default collections. Essentially, youCOMMENT
MEDIUMinternal/terraform/context.go506 return // should not happen, but we'll be robustCODE
MEDIUMinternal/terraform/context.go510 continue // should not happen, but we'll be robustCODE
MEDIUMinternal/terraform/context_apply_checks_test.go752 // Essentially we make a check block in a child module that depends on aCOMMENT
MEDIUMinternal/terraform/evaluate.go664 // TODO: When deferred actions are more stable and robust in stacks, itCOMMENT
MEDIUMinternal/stacks/stackconfig/parser/walker.go52 // we'll be robust about it nonetheless.COMMENT
MEDIUM…acks/stackruntime/internal/stackeval/terraform_hook.go133 // Weird, but we'll just tolerate it to be robust.COMMENT
MEDIUM…s/stackruntime/internal/stackeval/removed_component.go78// Essentially, a removed block can target components across multiple stackCOMMENT
MEDIUM…l/stacks/stackruntime/internal/stackeval/main_apply.go47 // it anyway just to be robust in case there's a bug further upCOMMENT
MEDIUMinternal/repl/session.go176 // but we'll be robust here and just pass through the GoStringCOMMENT
MEDIUMinternal/lang/functions.go339 // robust.COMMENT
MEDIUMinternal/lang/blocktoattr/fixup.go34 // in the presence of poorly-configured test mocks, so we'll be robustCOMMENT
MEDIUMinternal/lang/globalref/analyzer_meta_references.go364 // for all of the relative traversal types, but we'll be robust inCOMMENT
MEDIUMinternal/lang/funcs/datetime.go138 // be robust here.COMMENT
MEDIUMinternal/configs/experiments.go167 // to return any other error type, but we'll handle it to be robust.COMMENT
MEDIUMinternal/configs/configschema/validate_traversal.go69 // but we'll handle this with a generic error anyway to be robust.COMMENT
MEDIUMinternal/configs/configschema/empty_value.go59 // but we'll be robust and return a result nonetheless.COMMENT
MEDIUMinternal/configs/configload/loader_snapshot.go234 // This function doesn't try to be incredibly robust in supportingCOMMENT
MEDIUMinternal/states/state_test.go357 // happen" case but we'll test to make sure we're robust toCOMMENT
MEDIUMinternal/states/statemgr/filesystem.go127 // TODO: this should use a more robust method of writing state, by firstCOMMENT
MEDIUMinternal/states/statemgr/persistent.go83// Some implementations may optionally utilize config schema to persistCOMMENT
MEDIUMinternal/command/meta_providers.go162 // unit tests might not always populate Meta fully and so we'll be robustCOMMENT
MEDIUMinternal/command/cliconfig/cliconfig.go383 // improve on this later using the more-robust merging behaviorCOMMENT
MEDIUMinternal/command/views/hook_ui.go123 // bug in the caller but we'll ignore it in order to be robust.COMMENT
MEDIUMinternal/command/views/hook_ui.go451 // bug in the caller but we'll ignore it in order to be robust.COMMENT
MEDIUMinternal/command/clistate/local_state.go64// TODO: this should use a more robust method of writing state, by firstCOMMENT
MEDIUM…tdata/move-statement-implied/move-statement-implied.tf1# This fixture is useful only in conjunction with a previous run state thatCOMMENT
MEDIUM…move-statement-implied/child/move-statement-implied.tf1# This fixture is useful only in conjunction with a previous run state thatCOMMENT
MEDIUMinternal/moduletest/status.go33// Essentially, if a test suite has a bunch of failures and passes the overallCOMMENT
MEDIUMinternal/moduletest/mocking/fill.go75// directly to it or to any nested objects. Essentially, this is a "safe"COMMENT
MEDIUMinternal/moduletest/mocking/values.go88 // the validation, but we want this function to be robust and not panicCOMMENT
MEDIUMinternal/moduletest/mocking/values.go113 // This transform should be robust (in that it should never fail), theCOMMENT
MEDIUMinternal/moduletest/mocking/values.go114 // inner call to generateValue should be robust as well so it should alwaysCOMMENT
MEDIUMinternal/moduletest/mocking/values.go139 // gave us. getMockedDataForPath is robust, so even in an error itCOMMENT
MEDIUMinternal/moduletest/mocking/values.go142 // Now get the replacement value. This function should be robust inCOMMENT
LOWscripts/changelog-links.sh10# released items are presented as clickable links, but we can just use theCOMMENT
MEDIUM.github/workflows/build.yml208 # environment variables defined above. The e2e test harnessCOMMENT
Fake / Example Data67 hits · 67 pts
SeverityFileLineSnippetContext
LOWmain_test.go147 t.Setenv("OTHER_ENV", "placeholder")CODE
LOWinternal/terraform/context_apply2_test.go3410 resource "test_thing" "placeholder" {CODE
LOWinternal/stacks/stackruntime/plan_test.go6690 "email": cty.StringVal("user@example.com"),CODE
LOWinternal/stacks/stackruntime/plan_test.go6712 "email": cty.StringVal("user@example.com"),CODE
LOWinternal/stacks/stackruntime/plan_test.go6723 "email": cty.StringVal("user@example.com"),CODE
LOWinternal/stacks/stackruntime/plan_test.go6734 "email": cty.StringVal("user@example.com"),CODE
LOWinternal/stacks/stackruntime/plan_test.go6745 "email": cty.StringVal("user@example.com"),CODE
LOWinternal/stacks/stackruntime/plan_test.go6756 "email": cty.StringVal("user@example.com"),CODE
LOWinternal/stacks/stackruntime/plan_test.go6767 "email": cty.StringVal("user@example.com"),CODE
LOWinternal/plugin6/grpc_provider_test.go3272 data := []byte("Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod" +CODE
LOWinternal/plugin6/grpc_provider_test.go3272 data := []byte("Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod" +CODE
LOWinternal/backend/remote/testdata/plan-long-line/main.tf3 long_line = "[{'_id':'5c5ab0ed7de45e993ffb9eeb','index':0,'guid':'e734d772-6b5a-4cb0-805c-91cd5e560e20','isActive':fCODE
LOWinternal/cloud/state_test.go321 Token: "placeholder",CODE
LOWinternal/cloud/testing.go339 Token: "fake-token",CODE
LOWinternal/cloud/testdata/plan-long-line/main.tf3 long_line = "[{'_id':'5c5ab0ed7de45e993ffb9eeb','index':0,'guid':'e734d772-6b5a-4cb0-805c-91cd5e560e20','isActive':fCODE
LOWinternal/states/statemgr/statemgr_fake.go89 return "placeholder", nilCODE
LOWinternal/states/statemgr/statemgr_fake.go99 if id != "placeholder" {CODE
LOWinternal/states/statemgr/statemgr_fake.go140 return "placeholder", nilCODE
LOWinternal/command/apply_test.go1500 planPath := applyFixturePlanFileWithVariableValue(t, "lorem ipsum")CODE
LOWinternal/command/graph_test.go332 Type: "placeholder",CODE
LOWinternal/command/show_test.go1558 idVal = cty.StringVal("placeholder")CODE
LOWinternal/command/show_test.go1594 idVal = cty.StringVal("placeholder")CODE
LOWinternal/command/testdata/login-oauth-server/main.go65 client = "placeholder"CODE
LOWinternal/command/testdata/show-json/drift/output.json17 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/drift/output.json30 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/drift/output.json50 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/drift/output.json54 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/drift/output.json75 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/drift/output.json79 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/drift/output.json98 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/drift/output.json102 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/drift/output.json124 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/drift/output.json137 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/moved/output.json17 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/moved/output.json38 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/moved/output.json42 "id": "placeholder"CODE
LOWinternal/command/testdata/show-json/moved/output.json64 "id": "placeholder"CODE
LOW…nal/command/testdata/show-json/moved-drift/output.json17 "id": "placeholder"CODE
LOW…nal/command/testdata/show-json/moved-drift/output.json30 "id": "placeholder"CODE
LOW…nal/command/testdata/show-json/moved-drift/output.json51 "id": "placeholder"CODE
LOW…nal/command/testdata/show-json/moved-drift/output.json55 "id": "placeholder"CODE
LOW…nal/command/testdata/show-json/moved-drift/output.json76 "id": "placeholder"CODE
LOW…nal/command/testdata/show-json/moved-drift/output.json80 "id": "placeholder"CODE
LOW…nal/command/testdata/show-json/moved-drift/output.json100 "id": "placeholder"CODE
LOW…nal/command/testdata/show-json/moved-drift/output.json104 "id": "placeholder"CODE
LOW…nal/command/testdata/show-json/moved-drift/output.json126 "id": "placeholder"CODE
LOW…nal/command/testdata/show-json/moved-drift/output.json139 "id": "placeholder"CODE
LOW…testdata/show-json/conditions/output-refresh-only.json19 "value": "placeholder"CODE
LOW…testdata/show-json/conditions/output-refresh-only.json30 "after": "placeholder",CODE
LOW…testdata/show-json/conditions/output-refresh-only.json43 "value": "placeholder",CODE
LOW…testdata/show-json/conditions/output-refresh-only.json74 "id": "placeholder",CODE
LOW…al/command/testdata/show-json/basic-update/output.json29 "id": "placeholder"CODE
LOW…al/command/testdata/show-json/basic-update/output.json49 "id": "placeholder"CODE
LOW…al/command/testdata/show-json/basic-update/output.json53 "id": "placeholder"CODE
LOW…al/command/testdata/show-json/basic-update/output.json94 "id": "placeholder"CODE
LOW…al/command/testdata/show-json/basic-delete/output.json29 "id": "placeholder"CODE
LOW…al/command/testdata/show-json/basic-delete/output.json49 "id": "placeholder"CODE
LOW…al/command/testdata/show-json/basic-delete/output.json53 "id": "placeholder"CODE
LOW…al/command/testdata/show-json/basic-delete/output.json73 "id": "placeholder"CODE
LOW…al/command/testdata/show-json/basic-delete/output.json115 "id": "placeholder"CODE
7 more matches not shown…
Synthetic Comment Markers5 hits · 30 pts
SeverityFileLineSnippetContext
HIGHinternal/resources/ephemeral/ephemeral_resource_test.go88 // and at least one additional time as requested by the instance.COMMENT
HIGHinternal/configs/parser.go96// have been loaded through this parser, with source filenames (as requestedCOMMENT
HIGHinternal/command/jsonformat/plan.go566 buf.WriteString(fmt.Sprintf("[bold] # %s[reset] will be [bold][red]replaced[reset], as requested", dispAddr))CODE
HIGHinternal/command/jsonformat/plan_test.go6786 ExpectedOutput: ` # test_instance.example will be replaced, as requestedCODE
HIGHinternal/command/jsonformat/plan_test.go6797 ExpectedOutput: ` # test_instance.example will be replaced, as requestedCODE
Structural Annotation Overuse10 hits · 15 pts
SeverityFileLineSnippetContext
LOWinternal/terraform/context_apply_test.go2757 // Step 1: create the resources and instancesCOMMENT
LOWinternal/terraform/context_apply_test.go2792 // Step 2: update with an empty config, to destroy everythingCOMMENT
LOWinternal/providercache/installer.go212 // Step 1: Which providers might we need to fetch a new version of?COMMENT
LOWinternal/providercache/installer.go293 // Step 2: Query the provider source for each of the providers we selectedCOMMENT
LOWinternal/providercache/installer.go354 // Step 3: For each provider version we've decided we need to install,COMMENT
LOWinternal/command/workspace_command_test.go1557 // Step 1 - test list output with no diagnosticsCOMMENT
LOWinternal/command/workspace_command_test.go1597 // Step 2 - test list output with a warning diagnosticsCOMMENT
LOWinternal/command/workspace_command_test.go1656 // Step 3 - test that error diagnostics are shown in isolation (no additional output even if present)COMMENT
LOWinternal/tfdiags/diagnostic_extra.go319// NOTE: This function should be kept in sync with the extra interfaces definedCOMMENT
LOWscripts/goimportscheck.sh37# NOTE: The above intentionally excludes .pb.go files because those areCOMMENT
Slop Phrases6 hits · 15 pts
SeverityFileLineSnippetContext
MEDIUMinternal/terraform/context_plugins_test.go24// so the caller can feel free to modify the returned value to furtherCOMMENT
MEDIUMinternal/terraform/context_plugins_test.go60// callers can feel free to modify it once returned.COMMENT
MEDIUMinternal/command/testdata/fmt/general_out.tf3# here, but you can also add other _in.tf/_out.tf pairs in theCOMMENT
MEDIUMinternal/command/testdata/fmt/general_in.tf3# here, but you can also add other _in.tf/_out.tf pairs in theCOMMENT
LOW.github/workflows/enforce-changelog.yml1# This workflow makes sure contributors don't forget to add a changelog entry or explicitly opt-out of it.COMMENT
LOW.github/workflows/enforce-changelog.yml1# This workflow makes sure contributors don't forget to add a changelog entry or explicitly opt-out of it.COMMENT
Verbosity Indicators8 hits · 12 pts
SeverityFileLineSnippetContext
LOWinternal/terraform/context_apply_test.go2757 // Step 1: create the resources and instancesCOMMENT
LOWinternal/terraform/context_apply_test.go2792 // Step 2: update with an empty config, to destroy everythingCOMMENT
LOWinternal/providercache/installer.go212 // Step 1: Which providers might we need to fetch a new version of?COMMENT
LOWinternal/providercache/installer.go293 // Step 2: Query the provider source for each of the providers we selectedCOMMENT
LOWinternal/providercache/installer.go354 // Step 3: For each provider version we've decided we need to install,COMMENT
LOWinternal/dag/walk.go319 // Dependencies satisfied! We need to check if any erroredCOMMENT
LOWscripts/changelog.sh94 # We need to check if this is the first RC of the versionCOMMENT
LOWscripts/changelog.sh105 # We need to check if this is the first RC of the versionCOMMENT
TODO Padding2 hits · 3 pts
SeverityFileLineSnippetContext
LOWinternal/stacks/stackplan/planned_change_test.go1088 // TODO: Add test for "action invocation invoke trigger" when implementingCOMMENT
LOWinternal/command/arguments/metadata_functions.go37 // TODO: Add validation of unexpected arguments in future major version.COMMENT
Redundant / Tautological Comments2 hits · 3 pts
SeverityFileLineSnippetContext
LOWscripts/changelog.sh56 # Check if we already released this version alreadyCOMMENT
LOWscripts/changelog.sh188 # Check if yq is installedCOMMENT
Hyper-Verbose Identifiers2 hits · 2 pts
SeverityFileLineSnippetContext
LOWinternal/command/jsonformat/differ/attribute.go88 // function computeChangeForDynamicValues(), but external callers willCOMMENT
LOW.github/workflows/enforce-changelog.yml63 async function createOrUpdateChangelogComment(commentDetails, deleteComment) {CODE
Example Usage Blocks1 hit · 2 pts
SeverityFileLineSnippetContext
LOWinternal/tfdiags/compare.go13// Example usage:COMMENT