Skip to content

feat: workspace with the refuse-all pipeline, census and quality gates - #1

Closed
kvlonge wants to merge 8 commits into
mainfrom
feat/bootstrap
Closed

kvlonge wants to merge 8 commits into
mainfrom
feat/bootstrap

Conversation

@kvlonge

@kvlonge kvlonge commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Why

P0 of SQLQuack (CHI-831). It turns the May skeleton into the real workspace. Every construct is refused with a structured reason, and the quality gates block merges from the first commit.

Changes

  • Workspace crates: sqlquack-core (guarded parsing, classification, refusals, tree comparison, both re-parse checks), sqlquack-rules (empty catalogue), stub crates for snowflake, duckdb and compare, and the sqlquack facade with the census command.
  • Polyglot pinned to sqlbuild's revision, without transpile.
  • Gates: make check-ci (fmt, clippy -D warnings, the workspace lints, ruff, ty, fensu), a pinned toolchain, and CI with a cached dependency build.
  • Metadata: the crate repository URL and licence metadata are fixed.
  • Docs: a README, architecture notes and AGENTS.md.

Verification

  • make check-ci and make test pass locally, and the hygiene scan is clean.
  • A local census of a large private project parses all files, and its results match the earlier research.

kvlonge and others added 8 commits May 17, 2026 18:42
Replace the early skeleton with small crates: sqlquack-core (parsing,
construct classification, refusals, tree comparison and both re-parse
checks), sqlquack-rules (empty catalogue), sqlquack-snowflake,
sqlquack-duckdb and sqlquack-compare (empty), and sqlquack (pipeline,
census and command line). Polyglot is pinned without its transpile
feature.
make check-ci runs rustfmt, clippy with -D warnings, fensu with the Rust
rule pack, ruff and ty. CI runs check-ci and the tests with a cached
dependency build.
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