Skip to content
This repository was archived by the owner on Oct 31, 2026. It is now read-only.

feat!: stop shipping queries - #83

Merged
exKAZUu merged 3 commits into
mainfrom
feat/stop-shipping-queries
Oct 30, 2026
Merged

exKAZUu merged 3 commits into
mainfrom
feat/stop-shipping-queries

Conversation

@exKAZUu

@exKAZUu exKAZUu commented Oct 30, 2026 •

Copy link
Copy Markdown
Member

Why

The fork maintains only full parsing. Its queries (highlights.scm, injections.scm) were shipped but no longer maintained or tested, so a permitted tree-shape change could leave a published query referencing a node that no longer exists. No WillBooster product uses them. Removing them makes the published package match the maintenance scope and removes the agent rule for deleting broken queries.

Requirements

  • required: stop shipping queries in every tree-sitter repository, removing every reference to them (requester)
  • required: state the support scope in the README consistently with the agent instructions: full parsing maintained, incremental parsing provided as upstream but untested, no queries shipped (requester)
  • required: once this change passes review, apply the same wording to every tree-sitter repository (requester)
  • required: include every change that wbfy makes (requester)
  • chosen: a commit that changes the tree shape is marked breaking (!) so it is released as a new major version, which makes the README's statement about major versions true
  • chosen: the incremental-parsing code stays as in upstream, because it contributes almost nothing to the Wasm size (about 2 KB of a 28 MB ultra-uni build) and removing it would only add upstream merge conflicts

Customer Summary

Breaking: the npm package and the Rust crate no longer ship highlight or injection queries. tree-sitter.json no longer lists them, and the Rust constants HIGHLIGHT_QUERY_BLOCK, INJECTION_QUERY_BLOCK, HIGHLIGHT_QUERY_INLINE, and INJECTION_QUERY_INLINE are removed. The README gains a "Support Scope" section stating what this fork maintains.

Technical Summary

  • Deleted tree-sitter-markdown/queries/ and tree-sitter-markdown-inline/queries/, and their entries in package.json files, Cargo.toml include, tree-sitter.json, and bindings/rust/lib.rs.
  • AGENTS_EXTRA.md (and the regenerated AGENTS.md) now says the fork ships no queries and marks tree-shape changes as breaking; the rule for deleting broken queries is gone.
  • README: the Rust API paragraph lists both differences from upstream tree-sitter-md (no query constants, and MarkdownCursor::field_name borrowing from the tree).
  • wbfy also updated @types/bun, @willbooster/wb, oxlint-tsgolint, and bun.lock.

Testing

  • cargo test and bun run verify pass. code-gauge reports only existing violations in files this PR does not touch.
  • PR CI passed on a554874.

exKAZUu and others added 3 commits October 30, 2026 17:45
Co-authored-by: WillBooster (Claude Code) <agent@willbooster.com>
Co-authored-by: WillBooster (Claude Code) <agent@willbooster.com>
Co-authored-by: WillBooster (Claude Code) <agent@willbooster.com>
@exKAZUu exKAZUu self-assigned this Oct 30, 2026
@exKAZUu
exKAZUu merged commit ac1fb03 into main Oct 30, 2026
11 checks passed
@exKAZUu
exKAZUu deleted the feat/stop-shipping-queries branch October 30, 2026 09:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant