feat: add CLI flag consistency checker agentic workflow#494
Conversation
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Mossaka
left a comment
There was a problem hiding this comment.
Review: REQUEST_CHANGES
@copilot Please fix the following issues:
Issue 1: Non-existent File Reference
The workflow references AGENTS.md on lines 68-69:
# Check agent files
cat AGENTS.md
cat CLAUDE.mdAGENTS.md does not exist in this repository - only CLAUDE.md exists. Remove the AGENTS.md reference.
Issue 2: No CI Checks
No CI checks have run on this branch. Please ensure CI passes before this can be merged.
Suggestions (Optional)
- Consider monthly schedule instead of weekly to avoid discussion spam if issues persist
- Add logic to check for existing open Discussions with the same title prefix before creating duplicates
The overall feature is valuable - just needs these fixes.
Issue 1: Issue 2: Build and lint pass locally ( |
…orkflow-cli-check
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
✅ Coverage Check PassedOverall Coverage
Coverage comparison generated by |
|
Smoke Test Results - Claude Engine Last 2 merged PRs:
✅ GitHub MCP - PASS Overall Status: PASS
|
Smoke Test Results (Run #21686296501)Last 2 Merged PRs:
Tests:
Status: PASS 🎉 cc @Mossaka
|
There was a problem hiding this comment.
Pull request overview
This PR adds a new agentic workflow that performs weekly automated checks to identify documentation drift for AWF CLI flags. The workflow compares the CLI implementation against multiple documentation files and reports inconsistencies via GitHub Discussions.
Changes:
- Added
.github/workflows/cli-flag-consistency-checker.mddefining the workflow's instructions for the AI agent - Added
.github/workflows/cli-flag-consistency-checker.lock.ymlas the compiled GitHub Actions workflow configuration
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.github/workflows/cli-flag-consistency-checker.md |
Defines the AI agent's instructions for analyzing CLI flags across src/cli.ts and documentation files (docs/usage.md, cli-reference.md, README.md, AGENTS.md, CLAUDE.md), specifying the analysis steps, output format, and edge cases to handle |
.github/workflows/cli-flag-consistency-checker.lock.yml |
Auto-generated GitHub Actions workflow configuration compiled from the markdown file, scheduled to run weekly on Wednesdays, with proper permissions, tooling setup, and safe-output configuration for creating discussions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 2. **Usage documentation** (`docs/usage.md`) - User-facing documentation | ||
| 3. **CLI reference** (`docs-site/src/content/docs/reference/cli-reference.md`) - Detailed reference docs | ||
| 4. **README.md** - Quick start and overview documentation | ||
| 5. **AGENTS.md** and `CLAUDE.md` - Agent instruction files |
There was a problem hiding this comment.
Inconsistent markdown formatting for file names. AGENTS.md uses bold (AGENTS.md) while CLAUDE.md uses code backticks (CLAUDE.md). Both should use the same formatting style for consistency. Based on the other file references in this list (lines 25-28), code backticks should be used for both file names.
| 5. **AGENTS.md** and `CLAUDE.md` - Agent instruction files | |
| 5. `AGENTS.md` and `CLAUDE.md` - Agent instruction files |
|
|
||
| ### 4. Check for Subcommand Consistency | ||
|
|
||
| The CLI has subcommands (`logs`, `logs stats`, `logs summary`). Verify: |
There was a problem hiding this comment.
The description of subcommands is potentially misleading. The CLI has one command "logs" with two subcommands "stats" and "summary". The current wording "The CLI has subcommands (logs, logs stats, logs summary)" could be misinterpreted to suggest three separate commands. Consider clarifying this as "The CLI has a logs command with subcommands stats and summary" to accurately reflect the command hierarchy.
| The CLI has subcommands (`logs`, `logs stats`, `logs summary`). Verify: | |
| The CLI has a `logs` command with subcommands `stats` and `summary`. Verify: |
Adds a weekly agentic workflow that detects documentation drift for AWF CLI flags.
Changes
.github/workflows/cli-flag-consistency-checker.mdsrc/cli.tsagainstdocs/usage.md,cli-reference.md,README.md,AGENTS.mdcreate-discussionsafe-outputReport Structure
The generated Discussion includes:
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.