fix: consistency audit — feature_name convention, git:dev references, stale exits, agent ownership#167
Open
nullhack wants to merge 1 commit into
Open
fix: consistency audit — feature_name convention, git:dev references, stale exits, agent ownership#167nullhack wants to merge 1 commit into
nullhack wants to merge 1 commit into
Conversation
…ences, stale exits, missing templates, agent ownership - AGENTS.md: main→dev in branch discipline, planning-flow→define-flow example, feature_id→feature_name - Flow YAML files: feature_id→feature_name everywhere, params: [feature_name] in define-flow, removed unreachable exits (cancelled from develop-flow, needs-reinterview from define-flow) - Skills: feature_id→feature_name in write-bdd-features, select-feature, create-py-stubs; Available knowledge line added to setup-apply - Knowledge: feature_id→feature_name in test-stubs.md and source-stubs.md - Templates: new test stub template (.templates/tests/features/), 3 new cache templates (.templates/.cache/) - Agents: SE ownership/noqa/pyproject identity statement; implement-minimum fix-existing triggers; polish-code fix-all-violations - Removed stale @id references from AGENTS.md, README.md, docs/audit/; strengthened write-test anti-noise rules
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Full consistency audit fixes across AGENTS.md, flow definitions, skills, knowledge, and templates.
Changes
Convention fixes
feature_id→feature_name: Unified across 5 flow files, 3 skill files, 2 knowledge files. Matches the template filename convention.main→dev: Fixed stale branch references in AGENTS.md (lines 89, 147, 205) — all flows use onlygit: devandgit: feature.planning-flow→define-flow: Fixed stale example in AGENTS.md (line 99).Flow YAML fixes
params: [feature_name]todefine-flow.yamlcancelledexit fromdevelop-flow.yamlneeds-reinterviewexit fromdefine-flow.yaml(handled internally via subflow routing)Agent/Knowledge ownership
noqa, neverpyproject.tomlimplement-minimumskill: fix existing test/beehave failures in touched filespolish-codeskill: fix all violations codebase-widewrite-testskill: explicit anti-noise rules (no assertions/setup/teardown unrelated to Example intent)setup-applyskill: added missingAvailable knowledgelineTemplates
.templates/tests/features/<rule_slug>_test.py.template.templates/.cache/interview-notes/,sim/,acceptance/) — on disk only (gitignore excludes.cache/)<feature_id>variant from acceptance template directoryStale @id cleanup