Skip to content

Provision Go for all serena-go.md consumers, not just Linter Miner - #53383

Merged
pelikhan merged 2 commits into
mainfrom
copilot/deep-report-provision-go-for-typist
Aug 17, 2026
Merged

Provision Go for all serena-go.md consumers, not just Linter Miner#53383
pelikhan merged 2 commits into
mainfrom
copilot/deep-report-provision-go-for-typist

Conversation

Copilot AI commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

serena-go.md (Serena's Go LSP) is imported by 16 workflows, but only linter-miner.md carried a pre-agent-steps workaround installing Go before Serena startup. Every other consumer hits the LSP init failure (Go is not installed), forcing the agent to fall back to slow manual grep/awk analysis and burn through its AI credit budget — Typist just hit this exact failure.

Changes

  • Moved the actions/setup-go + go version preflight guard from linter-miner.md into the shared .github/workflows/shared/mcp/serena-go.md import, so it's applied once for all 16 consumers (imported pre-agent-steps are merged into the consuming workflow).
  • Removed the now-redundant duplicate Go setup steps from linter-miner.md, keeping only its workflow-specific preload step.
  • Recompiled the 16 affected .lock.yml files: archie, cloclo, daily-function-namer, developer-docs-consolidator, glossary-maintainer, linter-miner, mcp-inspector, purelock, sergo, smoke-codex, smoke-copilot-aoai-apikey, smoke-copilot-aoai-entra, smoke-copilot-arm, smoke-copilot, terminal-stylist, typist.
# .github/workflows/shared/mcp/serena-go.md
imports:
  - uses: ./serena.md
    with:
      languages: ["go"]
pre-agent-steps:
  - name: Setup Go
    uses: actions/setup-go@v7.0.0
    with:
      go-version-file: go.mod
      cache: true
  - name: Verify Go installation
    run: go version

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix provision of Go for all serena-go.md consumers Provision Go for all serena-go.md consumers, not just Linter Miner Aug 17, 2026
Copilot AI requested a review from pelikhan August 17, 2026 13:06
@pelikhan
pelikhan marked this pull request as ready for review August 17, 2026 13:11
@github-actions

Copy link
Copy Markdown
Contributor

Great work @Copilot! 👋 This fix elegantly solves the Serena Go LSP initialization failures across all 16 workflow consumers in one place, preventing the kind of credit-exhausting fallback that Typist just experienced.

What looks good:

This is well-targeted and focused. Ready for review! 🚀

Generated by ✅ Contribution Check · auto · 45.6 AIC · ⌖ 3.1 AIC · ⊞ 9.1K ·

Copilot AI balanced review requested due to automatic review settings August 17, 2026 13:11
@pelikhan
pelikhan merged commit 0c70219 into main Aug 17, 2026
@pelikhan
pelikhan deleted the copilot/deep-report-provision-go-for-typist branch August 17, 2026 13:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Centralizes Go provisioning for Serena’s Go LSP so consuming workflows initialize semantic analysis successfully.

Changes:

  • Adds shared Go setup and verification steps.
  • Removes Linter Miner’s duplicate setup.
  • Regenerates 16 direct consumer workflows.
Show a summary per file
File Description
.github/workflows/shared/mcp/serena-go.md Adds shared Go provisioning.
.github/workflows/linter-miner.md Removes duplicate Go setup.
.github/workflows/archie.lock.yml Regenerates workflow with Go setup.
.github/workflows/cloclo.lock.yml Regenerates workflow with Go setup.
.github/workflows/daily-function-namer.lock.yml Regenerates workflow with Go setup.
.github/workflows/developer-docs-consolidator.lock.yml Regenerates workflow with Go setup.
.github/workflows/glossary-maintainer.lock.yml Regenerates workflow with Go setup.
.github/workflows/linter-miner.lock.yml Updates generated metadata.
.github/workflows/mcp-inspector.lock.yml Adds Go setup and changes strict-mode output.
.github/workflows/purelock.lock.yml Regenerates workflow with Go setup.
.github/workflows/sergo.lock.yml Regenerates workflow with Go setup.
.github/workflows/smoke-codex.lock.yml Regenerates workflow with Go setup.
.github/workflows/smoke-copilot-aoai-apikey.lock.yml Regenerates workflow with Go setup.
.github/workflows/smoke-copilot-aoai-entra.lock.yml Regenerates workflow with Go setup.
.github/workflows/smoke-copilot-arm.lock.yml Regenerates workflow with Go setup.
.github/workflows/smoke-copilot.lock.yml Regenerates workflow with Go setup.
.github/workflows/terminal-stylist.lock.yml Regenerates workflow with Go setup.
.github/workflows/typist.lock.yml Regenerates workflow with Go setup.

Review details

  • Files reviewed: 18/18 changed files
  • Comments generated: 2
  • Review effort level: Balanced

GH_AW_INFO_CACHE_MEMORY: "true"
GH_AW_INFO_FRONTMATTER_EMOJI: "🔍"
GH_AW_COMPILED_STRICT: "false"
GH_AW_COMPILED_STRICT: "true"
Comment on lines +21 to +23
pre-agent-steps:
- name: Setup Go
uses: actions/setup-go@v7.0.0
Copilot AI added a commit that referenced this pull request Aug 17, 2026
…53383)

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[deep-report] Provision Go for all serena-go.md consumers, not just Linter Miner (Typist just hit the same credit-exhausting failure)

3 participants