feat(skills): add pre-filing version check to swamp-issue bug workflow#1377
Conversation
…w (swamp-club#321) Before filing a bug, the agent now checks if the user is on an outdated version and verifies whether the diagnosed bug was already fixed in the latest source. Converts the bug workflow to a state machine (gather_details → version_check → submit → verify) using the same Gate/Action/Verify/On Failure pattern as swamp-getting-started. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Move implementation details (lock internals, --skip-migration nuances, legacy layout migration) from SKILL.md to references/troubleshooting.md. Add verification steps after extension source add and lock release. Trim custom datastores/drivers sections. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Code Review
Blocking Issues
None.
Suggestions
- Lost
.swamp.yamlconfig snippet (swamp-repo/SKILL.md): The removed "Custom Datastores" section included a.swamp.yamlconfiguration example (type:/config:fields) that showed how third-party datastores are declared. The new one-liner points toswamp-extension, which covers creating extensions — but the YAML config example was useful for consuming them. Consider adding a brief snippet back to the SKILL.md or to a reference file. Not blocking sinceswamp help datastoreand the extension docs cover it.
Notes
CLAUDE.md compliance:
- SKILL.md files are uppercase ✓
- Frontmatter present with
name+description✓ - swamp-issue/SKILL.md: 231 lines (under 500 limit) ✓
- swamp-repo/SKILL.md: 402 lines (under 500 limit) ✓
- No extraneous files ✓
- Content correctly split between SKILL.md and
references/files ✓
State machine design (swamp-issue bug workflow): Well-structured Gate/Action/Verify/On Failure pattern. The defensive failure handling (all errors → inconclusive → proceed to submit) is exactly right — never blocks the user from filing. The "prefer inconclusive over bug_fixed" guidance in version_check.md is a good safety rail.
swamp-repo refactoring: Clean extraction of Lock Internals, --skip-migration details, and Legacy Extension Layout Migration into references/troubleshooting.md. The added verification steps (confirm source loaded after extension source add, verify lock cleared after release) are practical improvements.
DDD: No domain code changes — skill documentation only. No DDD concerns.
Security: No executable code changes. CLI commands referenced in the skill are standard swamp commands with no injection vectors.
Summary
gather_details → version_check → submit → verify) matching the swamp-getting-started patternversion_checkstate that checks if the user is on an outdated binary, fetches the latest source viaswamp source fetch, and has the agent re-read diagnosed files to determine if the bug was already fixedbug_fixed(tell user to update),bug_present(proceed to file),inconclusive(proceed to file) — never blocks filingreferences/version_check.mdwith the detailed procedureCloses swamp-club#321.
Test plan
swamp-issueskill to file a bug against swamp — verify the agent runs the version check state before submittingswamp update --check --jsonoutput is parsed correctlyswamp source fetch --version <latestVersion>works with the CalVer version string🤖 Generated with Claude Code