Skip to content

Make Archify audience-first before diagram authoring - #238

Closed
dum3r wants to merge 2 commits into
tt-a1i:mainfrom
dum3r:codex/audience-first-explanations
Closed

dum3r wants to merge 2 commits into
tt-a1i:mainfrom
dum3r:codex/audience-first-explanations

Conversation

@dum3r

@dum3r dum3r commented Aug 31, 2026

Copy link
Copy Markdown

Summary

  • freeze the intended reader, takeaway, and reader question before geometry
  • require causal branch labels, explicit parallel-source convergence, and progressive disclosure
  • add a reverse-reader gate so diagrams must stand alone without the author
  • preserve every existing Archify diagram type, viewer feature, validation, delivery, and export capability

Validation

  • skill-creator quick validation: pass
  • authoring safety contract: 4/4 pass
  • DITRA manager architecture: showcase 9/9, 0 errors, 0 warnings; delivery and visual-check pass at 1440x900, 1600x1000, 1920x1080, 2048x1320, light/dark captures visually reviewed
  • independent expense approval workflow: showcase 9/9, 0 errors, 0 warnings; delivery and the same visual-check matrix pass, light/dark captures visually reviewed
  • independent GLM 5.3 packet review: route proven, verdict pass, no findings
  • git diff --check: pass

Full npm test could not start in the clean clone because dev dependency simple-icons was not installed. No dependency installation was performed.

Production deployment: not run.

@tt-a1i tt-a1i left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the authoring improvement. Reviewed exact head a0acc9f.

Standards / package delivery

[P2] The published Skill change is not included in archify.zip. unzip -p archify.zip archify/SKILL.md yields the old 16041-byte document, while tracked source is 19212 bytes; the archive has no new Freeze the reader contract section. ZIP-installed users therefore receive none of the new reader-first instructions. CONTRIBUTING explicitly requires archive freshness for published SKILL.md changes. Rebuild the ZIP with canonical Node 22 after settling the source.

Spec

No other confirmed requirement mismatch in the new instructions.

Verification

Authoring/metadata focused tests pass; all 76 archive payload files compared after canonical package.json cleanup, and only SKILL.md differs. No source edits/merge; no current-head CI is a separate gate.

@dum3r dum3r closed this Sep 2, 2026
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.

2 participants