Skip to content

Expand evals to 25 and improve SKILL.md workflow guidance #32

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
CybotTM merged 3 commits into from
Apr 1, 2026
Merged

Expand evals to 25 and improve SKILL.md workflow guidance #32

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d0d056c
test: expand evals from 16 to 25 covering all skill areas
CybotTM Apr 1, 2026
bf57e26
feat: add workflow section and anti-fabrication principle to SKILL.md
CybotTM Apr 1, 2026
cf879a7
fix: replace fabricated phpunit command in root AGENTS.md
CybotTM Apr 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@
<!-- AGENTS-GENERATED:START commands -->
| Task | Command | ~Time |
|------|---------|-------|
| Test (all) | vendor/bin/phpunit | ~30s |
| Lint | `bash -n skills/agent-rules/scripts/*.sh` | ~5s |
| Validate | `skills/agent-rules/scripts/validate-structure.sh .` | ~10s |
<!-- AGENTS-GENERATED:END commands -->

> If commands fail, verify against Makefile/package.json/composer.json or ask user to update.
Expand Down
13 changes: 11 additions & 2 deletions skills/agent-rules/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,12 +40,21 @@ Generate and maintain AGENTS.md files following the [agents.md convention](https

See `references/scripts-guide.md` for full options.

## Workflow

1. **Detect**: `detect-project.sh` + `detect-scopes.sh` to identify stacks and subsystems
2. **Extract**: `extract-commands.sh`, `extract-ci-rules.sh`, etc. to gather facts
3. **Generate**: `generate-agents.sh` with `--style=thin` (default) or `--verbose`
4. **Verify**: `verify-content.sh` + `verify-commands.sh` -- MANDATORY before done
Comment on lines +45 to +48
Copy link

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

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

The workflow steps list script names without the scripts/ prefix and without a PATH argument, but the Scripts table above documents the invocations as scripts/<name>.sh PATH. For copy/paste correctness and consistency, update the workflow commands to use the same full paths and include a PATH placeholder (e.g., scripts/detect-project.sh PATH).

Suggested change
1. **Detect**: `detect-project.sh` + `detect-scopes.sh` to identify stacks and subsystems
2. **Extract**: `extract-commands.sh`, `extract-ci-rules.sh`, etc. to gather facts
3. **Generate**: `generate-agents.sh` with `--style=thin` (default) or `--verbose`
4. **Verify**: `verify-content.sh` + `verify-commands.sh` -- MANDATORY before done
1. **Detect**: `scripts/detect-project.sh PATH` + `scripts/detect-scopes.sh PATH` to identify stacks and subsystems
2. **Extract**: `scripts/extract-commands.sh PATH`, `scripts/extract-ci-rules.sh PATH`, etc. to gather facts
3. **Generate**: `scripts/generate-agents.sh PATH` with `--style=thin` (default) or `--verbose`
4. **Verify**: `scripts/verify-content.sh PATH` + `scripts/verify-commands.sh PATH` -- MANDATORY before done

Copilot uses AI. Check for mistakes.

Use `--update` to preserve human-curated content outside `<!-- GENERATED -->` markers.

## Core Principles

- **Structured over Prose** -- tables parse faster than paragraphs
- **Verified Commands** -- commands that don't work waste tokens
- **Never Fabricate** -- only document what exists; verify every command and path
- **Pointer Principle** -- point to files, don't duplicate content
- **Auto Symlinks** -- CLAUDE.md/GEMINI.md symlinks created by default (see [`ai-tool-compatibility.md`](references/ai-tool-compatibility.md))
- **Auto Symlinks** -- CLAUDE.md/GEMINI.md symlinks by default (see [`ai-tool-compatibility.md`](references/ai-tool-compatibility.md))

## References

Expand Down
Loading
Loading