Review: Convention: architecture component IDs for traceability labels
Verdict: READY
Re-review of board item #326. Previous verdict: NEEDS_REFINEMENT. Refinements posted as comment #6624.
Template Completeness
- [x] Type
- [x] Lineage
- [x] Repo
- [x] User Story
- [x] Context
- [x] File Targets (expanded in comment #6624 to include template-ticket modification)
- [x] Acceptance Criteria
- [x] Test Expectations
- [x] Constraints
- [x] Checklist
- [x] Related
File Targets
- [x]
convention-architecture-ids-- confirmed does not yet exist (pal-e-docs search returned empty). Creation is the deliverable. - [x]
template-ticket-- exists. Label Conventions table currently definesarch:{diagram-type}. Comment #6624 adds updating this table to scope. Verified the table still contains the conflicting definition (arch:domain, arch:dataflow, arch:deployment).
Repo Placement
OK. Issue filed on
forgejo_admin/pal-e-docs. Both deliverables (convention note creation, template-ticket update) are pal-e-docs notes. Correct repo.Dependencies
- Blocks #324 ("Enforce arch: + story: labels") -- that ticket needs this convention to define valid arch: values. Dependency documented in comment #6624.
- Board labels not yet updated. Item #326 labels =
type:feature,arch:kanban(missingblocks:324). Item #324 labels =type:feature,arch:kanban(missingdepends:326). This is an operational fix, not a scope issue. - No other blockers. Items #323 and #325 are independent.
Acceptance Criteria
All three criteria are verifiable by an agent after implementation:
- "Convention note exists with clear naming rules" -- agent can verify via
get_note(slug="convention-architecture-ids"). - "Examples cover all three diagram types" -- agent can read the note and check.
- "Pattern is consistent with existing label conventions" -- now resolvable because template-ticket update is in scope. After implementation, both documents will align on
arch:{component-name}.
Blast Radius
The semantic conflict between
template-ticket and convention-kanban-over-plans identified in the first review is now addressed by including the template-ticket update in scope. After implementation:convention-kanban-over-planssays "arch:X = which architecture component this touches" -- component granularity.template-ticketLabel Conventions will be updated fromarch:{diagram-type}toarch:{component-name}-- component granularity.- New
convention-architecture-idswill define the naming convention for component IDs. - All three documents will be consistent.
Current board usage (
arch:kanban on 4 agency items) already uses component-level granularity, consistent with the proposed convention. No migration needed for existing labels.Recommendation
Scope is solid. One operational task remains before moving to next_up:
- Add
blocks:324to item #326 labels anddepends:326to item #324 labels. This is a board hygiene step, not a scope change.