Skip to content

chore(MCPForge): workflow hygiene — ubuntu-24.04, permissions#135

Open
KooshaPari wants to merge 54 commits into
isaacphi:mainfrom
KooshaPari:chore/workflow-hygiene-ubuntu-24
Open

chore(MCPForge): workflow hygiene — ubuntu-24.04, permissions#135
KooshaPari wants to merge 54 commits into
isaacphi:mainfrom
KooshaPari:chore/workflow-hygiene-ubuntu-24

Conversation

@KooshaPari
Copy link
Copy Markdown

Summary

  • Replace ubuntu-latest with ubuntu-24.04 across all workflow files
  • Add minimal permissions: blocks to workflow files missing them

Test plan

  • CI passes on this branch
  • Merge after review

🤖 Generated with Claude Code

KooshaPari and others added 30 commits March 29, 2026 06:16
Co-authored-by: Claude Code <claude@anthropic.com>
- CLAUDE.md: project-level governance and mandate
- AGENTS.md: local agent contract and operating loop
- docs/worklogs/README.md: work audit template
- .github/workflows: quality-gate, fr-coverage, doc-links
- FUNCTIONAL_REQUIREMENTS.md: 7 FR stubs inferred from README
- tests/smoke_test: language-specific sanity check

Commit applies org standards to newly-cloned repos per batch-3 onboarding checklist.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Pin the top-level phenoShared CI reusable workflow to an immutable commit SHA instead of tracking main.

Validation:
- actionlint .github/workflows/ci.yml
- git diff --check

Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Add gopls check coverage to the common lint task so Taskfile-based validation includes the repo's Go LSP diagnostics without relying on the justfile audit target.

Co-authored-by: Codex <noreply@openai.com>
Remove generated snapshot diff files from integration test runs as part of the common clean task.

Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Avoid clearing the shared Go build cache from the common clean task; keep test-cache expiry and generated artifact cleanup.

Co-authored-by: Codex <noreply@openai.com>
Detected the repository as Go-based and kept the Taskfile common targets aligned with Go build, test, lint, and clean flows while centralizing the binary name.

Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Keep the common Taskfile test target on deterministic Go packages while leaving the real-filesystem watcher harness to its explicit package command. This keeps build, test, lint, and clean usable as a local common task set.\n\nValidation: task build test lint clean\n\nCo-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Detected Go tooling from go.mod and validated the common Taskfile tasks. Update clean so it clears both build and test caches before removing generated artifacts.

Co-authored-by: Codex <noreply@openai.com>
KooshaPari and others added 24 commits April 28, 2026 09:44
Use Go test-cache expiry for the clean task while keeping repo artifact cleanup intact. Avoid purging the shared Go build cache from this task because that can fail on active local cache directories.

Validated locally with task build, task test, task lint, and task clean.

Co-authored-by: Codex <noreply@openai.com>
Update the common clean task to clear both Go build and test caches so local build cache corruption can be repaired through the Taskfile workflow.

Co-authored-by: Codex <noreply@openai.com>
Add retry handling around Go cache cleanup so the common clean task tolerates transient cache directory races after tool invocations.

Co-authored-by: Codex <noreply@openai.com>
* chore: add standard issue templates

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* chore: add standard issue templates

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* chore: add standard issue templates

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* chore: add standard issue templates

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
* docs: add sladge badge

Co-authored-by: Codex <noreply@openai.com>

* chore: pin GitHub Actions to fixed SHAs

Pin GitHub Actions to immutable SHAs:
- checkout@v4: 34e114876b0b11c390a56381ad16ebd13914f8d5
- setup-go@v5: be666c2fcd27ec809703dec50e508c2fdc7f6654
- setup-node@v4: 48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e
- setup-python@v5: a26af69be951a213d495a4c3e4e4022e16d87065

* chore: pin GitHub Actions to specific SHAs

---------

Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Phenotype Agent <agent@phenotype.ai>
Pin all action refs to immutable SHAs across workflow files:
- checkout@v4 → @11bd71901bbe5b1630ceea73d27597364c9af683
- checkout@v6 → @de0fac2e4500dabe0009e67214ff5f5447ce83dd
- setup-node@v4/v5, setup-python@v4/v5, setup-go@v5
- upload-artifact@v4/v7, download-artifact@v4
- cache@v3/v4, github-script@v7
- configure-pages@v5/v6, deploy-pages@v4/v5
- upload-pages-artifact@v3/v5, dependency-review-action@v4

Fixes version-tag normalization (add v4/v5 tags where missing).
Fixes double-SHA corruption artifacts from prior patching rounds.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
- Add CLAUDE.md with Go MCP language server conventions
- Add .trufflehog.yml secrets scanning config

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
…p-go

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 28, 2026 08:30
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

2 participants