Experimental Command Gating with Visibility Flag · Entire

Review of Code Changes

Review the code changes against the base branch 'main'. The merge base commit for this comparison is 21a7b10fc4e89007b08637df116d8aeb834ded45. Run git diff 21a7b10fc4e89007b08637df116d8aeb834ded45 to inspect the changes relative to main. Provide prioritized, actionable findings.

Findings

{
  "findings":[
    {
      "title":"[P1] Remove unused staticcheck suppressions",
      "body":"With `nolintlint` enabled, these new `//nolint:staticcheck` comments fail lint because staticcheck does not emit a diagnostic for these in-module deprecated generated fields. Running golangci-lint against the patch reports this line plus the four new `CreatedMirror.Empty` suppressions in `repo_mirror.go` and `repo_mirror_create_wizard.go`, so CI lint will fail until the unnecessary directives are removed.",
      "confidence_score":0.97,
      "priority":1,
      "code_location":{
        "absolute_file_path":"/Users/gtrrz-victor/wks/cli/cli/cmd/entire/cli/cell_fanout.go",
        "line_range":{
          "start":106,
          "end":106
        }
      }
    }
  ],
  "overall_correctness":"patch is incorrect",
  "overall_explanation":"The code compiles and tests pass, but the patch introduces unused nolint directives that are rejected by the configured lint rules, so CI lint would fail.",
  "overall_confidence_score":0.96
}

Commits

• c5f5f27 docs: design for experimental command gating
• 8584dac docs: rename gate to Visible/IsVisible for clearer semantics
• 8622db2 feat: gate experimental commands behind build-time visibility flag

Description of Experimental Command Gating

Introduce cmd/entire/cli/experimental: a build-time gate (Visible, stamped false by GoReleaser via ldflags) that shows a curated set of maturing commands in developer builds and hides them in shipped releases. When visible, they are filed under a single 'Experimental commands:' cobra group in help; when hidden they carry no GroupID so release help is unchanged and cobra never references an unregistered group.

Gated commands include: tokens, import, review, investigate, blame, why, search, experts, runner (root) and checkpoint policy. Infra/plumbing, deprecated shortcuts, and aliases are untouched and stay always-hidden.