Add Arm enablement workflow and Codex skill#110
Conversation
brikin01
left a comment
There was a problem hiding this comment.
Thanks for putting this together—I left a few comments, though some are minor, so please use your best judgment. I’m not an expert in this type of analysis, so happy to be corrected if I’ve misunderstood anything
| @@ -0,0 +1,107 @@ | |||
| <!-- Place this prompt file at ~/.codex/prompts/arm-enablement.md to enable it. | |||
There was a problem hiding this comment.
Custom prompt files are now deprecated for Codex and are not invokable in more recent versions of Codex CLI. Can we convert this into a skill file?
There was a problem hiding this comment.
Agreed and I converted the legacy codex custom prompt into an arm-enablement skill with SKILL.md and agents/openai.yaml, including the arm mcp dependency. I also added installation and invocation guidance. I tested the skill with Codex 0.144.2 against bbolt and generated both report and artifacts successfully.
There was a problem hiding this comment.
Pull request overview
Adds an Arm enablement integration across VS Code prompts and Codex skills so users can run an Arm MCP–backed readiness assessment and generate reproducible Markdown/PDF reports with an explicit “report-only by default” posture.
Changes:
- Introduces a new VS Code chat prompt (
/arm-enablement) that guides Arm MCP scanning, evidence capture, and report/PDF generation. - Adds a Codex
arm-enablementskill (SKILL.md +agents/openai.yaml) that formalizes scope, scanning, container verification, and reporting expectations. - Documents Codex skill installation and invocation steps in the Arm MCP installation guide.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| agent-integrations/vs-code/arm-enablement.prompt.md | New VS Code prompt to drive Arm readiness assessment and produce Markdown/PDF artifacts. |
| agent-integrations/codex/arm-enablement/SKILL.md | New Codex skill instructions defining workflow, evidence requirements, and PDF export guidance. |
| agent-integrations/codex/arm-enablement/agents/openai.yaml | Codex agent metadata + dependency declaration for the Arm MCP server. |
| agent-integrations/agent-install-instructions.md | Adds steps to install/discover/invoke the new Codex skill. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
brikin01
left a comment
There was a problem hiding this comment.
Thanks for all the changes, looking really good! I just added a few more comments, most important one is about the MCP dependency block (I think we can just delete). The others (along with the comments Copilot generated) are fairly minor, feel free to treat those as non-blocking if you think they're not necessary.
Also don't worry about the failing integration test, it is unrelated to your change and I am fixing that in a separate PR.
|
Addressed all the comments and made a fresh commit accordingly !! Thanks |
brikin01
left a comment
There was a problem hiding this comment.
LGTM thanks for all the changes and for adding this into the repo!
Summary
Adds a reusable Arm enablement workflow for OSS repositories in VS Code and Codex.
The Codex integration is packaged as an
arm-enablementskill. It uses the Arm MCP Server to assess source, builds, dependencies, CI, and container supply chains, then creates evidence-backedarm-enablement-report.mdandarm-enablement-report.pdfartifacts. Report-only mode is the default; fixes and commits require explicit user requests.What changed
arm-enablementskill withagents/openai.yamlinterface metadata./skills, invocation with$arm-enablement, and the/workspacemount requirement./workspacescans for changed checkouts and reservegit_repofor an initial remote report-only scan.Validation
$arm-enablementfrom$HOME/.agents/skills, connected through the documented manual Arm MCP configuration, and assessed pinnedetcd-io/bboltcommitfa7ad3a72579c8766a84f7e765a4a9de997e0cf6in report-only mode.arm-enablement-report.mdandarm-enablement-report.pdf. The final PDF is 11 pages; every page was rendered and checked for clipping, overlap, blank pages, duplicate titles, table readability, and pagination.aarch64Ubuntu host,armlimited/arm-mcp:latestresolved to Arm64 digestsha256:d40b728c7f060915879bbd64a7ff25bf2ba77e44876f19775c21e8d25b681b76. Arm MCP Server 3.4.4 exposed all seven expected tools, and the Go scan completed with return code 0 and zero findings.linux/arm64. Its documented short test matrix and short race matrix passed for both freelist implementations. The temporary AWS checkout was clean after removing the MCP invocation-log sidecar.CI
integration-tests (ubuntu-24.04, linux/amd64)passed on commit3704576.integration-tests (ubuntu-24.04-arm, linux/arm64)passed on commit3704576.