Skip to content

fix: prefix agent filename with ado to distinguish from gh-aw#299

Merged
jamesadevine merged 2 commits intomainfrom
devinejames/gh-aw-agent-fixes
Apr 22, 2026
Merged

fix: prefix agent filename with ado to distinguish from gh-aw#299
jamesadevine merged 2 commits intomainfrom
devinejames/gh-aw-agent-fixes

Conversation

@jamesadevine
Copy link
Copy Markdown
Collaborator

Renames the init agent file from ado-aw.agent.md to ado.ado-aw.agent.md so it is clearly distinct from gh-aw agent files that share the same .github/agents/ directory.

@jamesadevine jamesadevine force-pushed the devinejames/gh-aw-agent-fixes branch from 019f11b to 08763fe Compare April 22, 2026 07:58
@jamesadevine jamesadevine merged commit b3ecca7 into main Apr 22, 2026
@jamesadevine jamesadevine deleted the devinejames/gh-aw-agent-fixes branch April 22, 2026 08:01
@github-actions
Copy link
Copy Markdown
Contributor

🔍 Rust PR Review

Summary: Looks good — clean, well-scoped rename with no missed references.

Findings

✅ What Looks Good

  • All five reference sites updated atomically: src/init.rs constant, AGENTS.md, README.md, and both integration test assertions in tests/init_tests.rs. No stale references to ado-aw.agent.md remain in the codebase.
  • detect.rs (which scans repos for compiled pipelines) correctly operates on the # @ado-aw source= header comment in YAML files, not on the init-agent filename — so it needs no changes.
  • src/data/init-agent.md (the template deployed to consumer repos by init) uses versioned v{{ compiler_version }} URLs, which is correct and unaffected by this rename.
  • The repo's own .github/agents/ado-agentic-workflows.agent.md uses main-branch URLs, appropriate for internal contributor use as opposed to the versioned template.
  • Separation of the repo's ado-aw and gh-aw dispatcher agents into distinct files (ado-agentic-workflows.agent.md vs. agentic-workflows.agent.md) is a sensible organisational improvement.

Generated by Rust PR Reviewer for issue #299 · ● 463.5K ·

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.

1 participant