docs: pure-OSS markdown, drop Nextra site#5
Merged
Conversation
DQL is a GitHub-native open-source project — no hosted docs site. All
documentation now lives as plain markdown in docs/, renders directly on
github.com, and is linked from the root README and per-package READMEs.
- Port 27 MDX pages to docs/{01-quickstart,02-concepts,03-install,guides,reference,architecture,contribute}
- Delete apps/docs/ (Nextra app) and .github/workflows/deploy-docs.yml
- Strip docs.duckcode.ai references from CLI banner, package READMEs,
Homebrew formula template, plugin-api JSDoc, and CI comments
- Write docs/README.md as a numbered reading-order index
- Rewrite root README.md to link into ./docs/ via relative paths
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
- docs job pointed at deleted apps/docs/scripts/link-check.mjs → replace with scripts/check-doc-links.mjs that validates every relative markdown link under docs/ and root README.md - format-check built only dql-cli (pnpm -F) which skipped deps and broke on missing apps/dql-notebook/dist → switch to full pnpm build (turbo handles dependency order) - apps/desktop tauri build required icons/ (not in repo) and raced with dql-ui writes during concurrent turbo tasks → make desktop's default build a no-op; tauri compile now lives only under release-desktop.yml (build:tauri script preserved for local/manual use) - declare dql-notebook-app as a desktop workspace dep so its build is ordered before desktop even after the no-op Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
formatSQLQuery() stripped all leading whitespace via .trim() on the raw SQL, which erased the common leading indent of the first line. Every fmt pass then prepended the block-level indent on top, so each run added ~2 spaces and the file drifted forever. `dql fmt --check` therefore always failed on any block with a multi-line query. Strip only leading newlines and trailing whitespace, compute the min leading whitespace across non-empty lines, and subtract it before re-indenting. Now `fmt` is a fixed point. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Two follow-ups to the previous fmt fix: 1. formatCall() was joining every SQL line with `,` as if each were a separate positional arg, mangling multi-line SELECTs in dashboards. Treat the whole SQL query as one chunk; add the inter-arg comma only after its last line. 2. formatSQLQuery() still drifted when the first SQL line shared a row with the opening `(` (e.g. `chart.bar(SELECT ...`). Its leading indent is lost by the parser, so including it in the dedent baseline made minLeading = 0 and every pass re-added the block indent. Use tail lines for the dedent baseline; fall back to the first line only when there are no others. Verified idempotent on both template files. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
KKranthi6881
added a commit
that referenced
this pull request
Apr 23, 2026
…r abstraction) Closes the v1.2.1 gap audit — 9 fixes, ordered by severity. No behavior changes beyond the #2 email-delivery correction; the rest add logging, ignore patterns, and a provider-extension point. Critical: - #1 ignore .dql/runs/ and .dql/cache/ (root + `dql init` scaffold) - #2 email notifier returns delivered:false in stub mode (was lying) Bugs: - #3 GET /api/blocks/body endpoint for bound-cell reloads - #5 /api/schema returns 500 + fallback instead of silent file-only - #6 log malformed scheduler run records to stderr - #7 replace empty catches in runtime/html emitters with console.warn Smells: - #8 move LLM providers into providers/; add LLMProvider type alias - #9 `dql schedule stop` via pidfile #4 was a false-positive — bound-cell lineage is already wired via the manifest builder's pathToBlockName lookup; comment updated. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
docs/(quickstart, concepts, install, guides, reference, architecture, contribute).apps/docs/(Nextra app) and.github/workflows/deploy-docs.yml; scrub everydocs.duckcode.aireference across READMEs, CLI banner, Homebrew template, plugin-api JSDoc, and CI comments.docs/README.mdis a numbered reading-order index; rootREADME.mdlinks into./docs/via relative paths.Test plan
grep -r docs.duckcode.aireturns nothing outside.git/ls apps/docsfails (directory deleted)docs/README.mdrenders correctly on github.com with working links to 01-quickstart / 02-concepts / 03-install and every guide/reference/architecture/contribute pageREADME.mdlinks into./docs/work on github.compnpm -r buildstill succeeds (no broken workspace references)🤖 Generated with Claude Code