Skip to content

docs(knowledge): approve reconciled zsh-lint parser guidance (#803) - #805

Merged
ss-o merged 1 commit into
mainfrom
feature-803-parser-record
Oct 16, 2026
Merged

ss-o merged 1 commit into
mainfrom
feature-803-parser-record

Conversation

@ss-o

@ss-o ss-o commented Oct 16, 2026

Copy link
Copy Markdown
Member

Summary

  • Update only the z-shell/zsh-lint parser front-end record in knowledge/project-delivery.json: revision f7ec572de9874ebd815fc4e49ce1419084348728 (docs(knowledge): reconcile zsh-lint parser guidance with f1c43fa (#803) #804), source blob dd238342996c67a36610d9fcf89b81645b225474, tested project revision f1c43fac9c8eadfc914a4af89a1beea11afca891 (zsh-lint PR 699), the consumer blob currently at that revision 59c28b6a2d0171de0187b04a9dce6de29fe37e8e, and content digest 75b347cd57824b0b99e9f9da62f61f501e328eb0d32da909d405d4b9aff9532f.
  • The Go AST and wiki records, targets and selectors are unchanged.
  • zsh-lint adopts this record in PR 699 by repinning Org Routing to this change's merge commit and regenerating .github/instructions/parser-front-end.instructions.md.

Closes #803

Impact review

  1. Classification: approval record for scoped project guidance, not shared policy, runtime behavior or enforcement.
  2. Consumers: z-shell/zsh-lint only, once its Org Routing caller pins a revision that contains this record.
  3. Owner: unchanged; the organization source stays editable and the project consumer stays generated.
  4. Duplication or contradiction: none added. The tooling index still ends its adoption paragraph with a pre-existing sentence saying no project delivery record is active for these sources; that predates this change and is left for separate follow-up.
  5. Manifests: no route changes; the record's target and selector match the existing z-shell/zsh-lint surface.
  6. Runtime delivery: the regenerated consumer keeps its native frontmatter and selector; runtime discovery in the project is checked after adoption, not here.
  7. Generation and size: knowledge-delivery.py --check passes for 57 organization consumers; the rendered project consumer differs in 5 lines from the one at the tested revision.

Verification

  • org-routing.py validate and org-routing.py verify-approved (with the agent-skills source root) pass, so the revision is an ancestor of this head and the source blob, rendered digest and selector match.
  • Applying this record to an export of zsh-lint f1c43fac reports AGENTS.md is current and updates only .github/instructions/parser-front-end.instructions.md, whose SHA-256 equals the recorded digest; org-routing.py check then passes. The contract link renders as ../../docs/project/parser-gap-workflow.md, the ADR links as immutable links at f7ec572d, and the provenance comment carries the new fields.
  • Markdownlint 0.49.1 with zsh-lint's configuration reports no findings on the regenerated consumer.
  • knowledge-delivery.py --check, knowledge-coverage.py and validate-agent-policy.py pass. 339 unit tests pass: knowledge coverage 7, delivery 14, decision records 17, agent policy 99, routing and policy report 91, Zsh standard policy 111.
  • git diff --check and trunk check --no-fix --ci on the manifest pass.

Agent handoff

Pin the z-shell/zsh-lint parser front-end delivery record to the source
merged in f7ec572 (blob dd23834), tested project revision f1c43fa, the
consumer blob currently at that revision (59c28b6) and the digest of the
complete rendered consumer. The Go AST and wiki records are unchanged.

Rendering with the tested revision makes the contract link local to the
consumer. Applying the record to an export of f1c43fa changes only
.github/instructions/parser-front-end.instructions.md, and check passes.

Refs #803
@ss-o
ss-o requested a review from a team as a code owner October 16, 2026 22:09
@ss-o ss-o added type:docs Documentation-only work. area:docs Documentation systems, content, or publishing. labels Oct 16, 2026
@ss-o ss-o self-assigned this Oct 16, 2026
@ss-o ss-o moved this to In Review in Z-shell Delivery Oct 16, 2026

@ss-o ss-o left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fallback review under ADR-0026: maintainer elected after the Copilot request did not register, on 4e9d705

Findings

No actionable findings in the one-file diff against main f7ec572 (merge base equal to base).

Note

This is an author-side review of record, not independent human approval. Adopting the record in zsh-lint PR 699, merging that PR and any release remain separate decisions.

Checklist verdicts

  • Scope: pass. Only the z-shell/zsh-lint parser front-end record changes, in five fields; its repository, source, target and the Go AST and wiki records are byte-identical. JSON formatting is unchanged.
  • Provenance: pass. revision is the #804 merge commit, an ancestor of this head; source_blob dd238342996c67a36610d9fcf89b81645b225474 is that commit's regular-file blob of the source. project_revision f1c43fac9c8eadfc914a4af89a1beea11afca891 is the revision the source was reconciled against in #804, and project_source_blob 59c28b6a2d0171de0187b04a9dce6de29fe37e8e is the consumer blob at that revision. verify-approved passes on this head, covering ancestry, blob identity, rendered digest and selector.
  • Delivery compatibility: pass. Applying the record to an export of zsh-lint f1c43fac reports AGENTS.md current and rewrites only .github/instructions/parser-front-end.instructions.md; the result's SHA-256 equals content_sha256, and check passes. Five lines change against the current consumer: the provenance comment, the namespace-neutral ownership and import-prefix sentences with the ADR-0041 link, and the two ADR links now pinned at f7ec572. The contract link stays consumer-relative. Markdownlint 0.49.1 with the project configuration reports no findings.
  • Verification: pass. Knowledge delivery, coverage, agent policy and routing validation pass locally with 339 unit tests, and git diff --check and Trunk pass on the manifest. All 11 hosted checks pass on this head, including Org Routing and Agent Instruction Validation; the commit signature is verified.

Limits and follow-up

f1c43fac is the head of unmerged zsh-lint PR 699; the record names it as the tested revision, and a later squash merge will not make it part of zsh-lint's main history. The tooling index's pre-existing sentence about no active project delivery records is outside this change. Native runtime discovery of the regenerated consumer is checked after adoption in the project. Review threads and comments were checked before publication; there are none.

@ss-o
ss-o merged commit f4ce168 into main Oct 16, 2026
11 checks passed
@ss-o
ss-o deleted the feature-803-parser-record branch October 16, 2026 22:13
@github-project-automation github-project-automation Bot moved this from In Review to Done in Z-shell Delivery Oct 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:docs Documentation systems, content, or publishing. type:docs Documentation-only work.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[docs]: reconcile zsh-lint parser guidance for installable packaging

1 participant