Skip to content

docs: refresh shell AST skill - #12

Merged
aliou merged 1 commit into
feat/updatesfrom
docs/sh-ast-skill
Aug 23, 2026
Merged

docs: refresh shell AST skill#12
aliou merged 1 commit into
feat/updatesfrom
docs/sh-ast-skill

Conversation

@aliou

@aliou aliou commented Jul 29, 2026

Copy link
Copy Markdown
Owner

Summary

Bring skills/sh-ast/SKILL.md in line with what the parser actually does.

  • Document SelectClause, CoprocClause, TimeClause, BraceExp, and ExtGlob.
  • Add a positions section (pos/end, Pos, NO_POS).
  • Document recoverErrors, parseStmtsSeq, parseWordsSeq, and splitBraces, with a brace-expansion example.
  • Replace stale limitations (positions, extglobs, arithmetic strings) with the ones that still hold.
  • Patch changeset for the bundled skill.

Replaces #5, which was written on a remote branch.

Verification

Every code snippet and node-type claim in the skill was exercised with a throwaway vitest file (brace example produces a BraceExp with 3 elements, seq helpers yield 3 statements / 3 words, recoverErrors returns errors, select/time/coproc parse to their nodes). That file was removed before committing.

pnpm test (20 files, 261 tests) and pnpm typecheck pass.

@aliou
aliou force-pushed the docs/sh-ast-skill branch from 81c7ebf to bdb95ef Compare July 30, 2026 06:33
@aliou
aliou changed the base branch from main to feat/updates July 30, 2026 06:34
@378-kaiabot
378-kaiabot Bot force-pushed the docs/sh-ast-skill branch from bdb95ef to 0a04d89 Compare August 22, 2026 12:00
@aliou
aliou merged commit b07ada4 into main Aug 23, 2026
1 of 2 checks passed
@aliou
aliou deleted the docs/sh-ast-skill branch August 23, 2026 12:22
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