Skip to content

docs(site): fix FileGlobMatch — gate.js globMatch, not python3 fnmatch#837

Merged
jamesadevine merged 1 commit into
mainfrom
docs/fix-filter-ir-fileglobmatch-python3-1780451012-68533f7a7e715717
Jun 3, 2026
Merged

docs(site): fix FileGlobMatch — gate.js globMatch, not python3 fnmatch#837
jamesadevine merged 1 commit into
mainfrom
docs/fix-filter-ir-fileglobmatch-python3-1780451012-68533f7a7e715717

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Corrects a stale and wrong claim in the Filter IR reference: FileGlobMatch was described as using "python3 fnmatch", but the actual evaluator is the TypeScript/Node.js gate.js bundle which uses its own globMatch() function — Python is not involved.
  • Adds a cross-reference link to the ado-script reference in the "Bundled Gate Evaluator" section, giving readers a direct path to the full design and build-pipeline docs.

Changes

  • site/src/content/docs/reference/filter-ir.mdx: corrected FileGlobMatch "Bash Shape" entry; added ado-script reference link.

Accuracy checks

  • Verified scripts/ado-script/src/gate/predicates.ts: both glob_match (case "glob_match") and file_glob_match (case "file_glob_match") are handled by the JavaScript globMatch() function — no Python code anywhere in the gate evaluator.
  • src/compile/filter_ir.rs line 173 comment: fnmatch(value, pattern) refers to the semantic shape, not a Python call; gate.js implements the same semantics via a custom regex.

Validation

  • cd site && npm ci && npm run build
  • Build succeeded: 33 page(s) built
  • Starlight links validator: "All internal links are valid."

Created by the docs-writer workflow.

Generated by Docs Writer · sonnet46 1.7M ·

- Corrected the 'Bash Shape' entry for FileGlobMatch from 'python3 `fnmatch`'
  to 'gate.js `globMatch`'; the evaluator is TypeScript/Node.js, not Python.
- Added cross-reference link to the ado-script reference page in the
  'Bundled Gate Evaluator' section.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jamesadevine jamesadevine marked this pull request as ready for review June 3, 2026 07:51
@jamesadevine jamesadevine merged commit f38c515 into main Jun 3, 2026
@jamesadevine jamesadevine deleted the docs/fix-filter-ir-fileglobmatch-python3-1780451012-68533f7a7e715717 branch June 3, 2026 07:51
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