Skip to content

fix(app): artifact.backlink refuses a malformed --change path by form (change 0460) - #336

Merged
danielhanold merged 5 commits into
mainfrom
fix/artifact-backlink-refuses-an-absolute-change-path-with-unkno
Sep 26, 2026
Merged

danielhanold merged 5 commits into
mainfrom
fix/artifact-backlink-refuses-an-absolute-change-path-with-unkno

Conversation

@danielhanold

Copy link
Copy Markdown
Owner

↩ Change 0460 — artifact.backlink refuses an absolute --change path with unknown-change

↩ Change 0460 — artifact.backlink refuses an absolute --change path with unknown-change

Summary

  • artifact.backlink now validates --change as a canonical repository-relative path, the same rule --artifact and change.attach-plan/attach-results enforce: absolute → absolute-path; empty, .. escape, or non-canonical spelling → path-escape. Messages name the flag and the expected form. Only a well-formed path matching no record still returns unknown-change. No writes on refusal.
  • docket-implement-next SKILL.md names the repo-relative form at both artifact.backlink call sites; embedded assets regenerated; one sentence trimmed to stay within the skill's word budget.

Review

Whole-branch review (docket-review-deep): clean, no findings.

Build evidence

command: go run ./cmd/docket development test
result: green
head_sha: 2bf5857
ran_at: 2026-09-26T12:40:06Z

Docket-Plan-Path: docs/superpowers/plans/2026-09-26-artifact-backlink-refuses-an-absolute-change-path-with-unkno.md
…under the word budget (change 0460)

Commit 379e409 named the repo-relative form on both artifact.backlink
flag placeholders, pushing skills/docket-implement-next/SKILL.md to 8029
words against its 8025-word TestSkillSizeBudgets ceiling. The plan-writer
payload sentence listed the synchronized change-file path twice; fold the
second mention into the first as the plan backlink's target. No meaning
lost, repo-relative wording kept, ceiling unchanged; embedded copy
regenerated.
@danielhanold
danielhanold merged commit ef9524c into main Sep 26, 2026
1 check passed
@danielhanold
danielhanold deleted the fix/artifact-backlink-refuses-an-absolute-change-path-with-unkno branch September 26, 2026 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant