Summary
Cosmetic: the skill changelog mis-interpolates the reviewer name, rendering "(approved by approved)" instead of the actual reviewer.
Evidence (dogfood campaign)
- The landed
SKILL_CHANGELOG.md reads:
- Write the progress.md handoff FIRST, before deep implementation (approved by approved)
- The reviewer-name field is interpolated with the action ("approved") instead of the reviewer identity.
Why it matters
Trivial, but it makes the changelog audit trail meaningless for attribution.
Proposed fix
Interpolate the actual reviewer name (or "human"/operator identity) into the changelog entry instead of the action verb.
Acceptance criteria
Source: dogfood/ITERATION_REPORT.md MINOR-10; dogfood/FLYWHEEL.md changelog excerpt.
Summary
Cosmetic: the skill changelog mis-interpolates the reviewer name, rendering "(approved by approved)" instead of the actual reviewer.
Evidence (dogfood campaign)
SKILL_CHANGELOG.mdreads:Why it matters
Trivial, but it makes the changelog audit trail meaningless for attribution.
Proposed fix
Interpolate the actual reviewer name (or "human"/operator identity) into the changelog entry instead of the action verb.
Acceptance criteria
Source:
dogfood/ITERATION_REPORT.mdMINOR-10;dogfood/FLYWHEEL.mdchangelog excerpt.