xavier/s3-vedrfolnir
Watch
1
0
Fork
You've already forked s3-vedrfolnir
 0
Code Pull requests Packages 1 Activity Actions

feat(v1.1.1): SQL highlight fix, autocomplete ext, dev tag + CI PR-trigger #4

Merged
xavier merged 1 commit from dev into main 2026-08-14 09:46:36 +02:00
Conversation 0 Commits 1 Files changed 26 +1644 -187
xavier commented 2026-08-14 09:46:31 +02:00
Owner
Copy link

Four-item patch on the v1.1 line (architect design in docs/architecture/v1.1.1-design.md; ADRs #34 + #35 appended).

  1. Restore SQL syntax highlighting — @codemirror/view de-duplicated to a single instance via a pnpm override (6.7.1 pin + 6.43.8 transitive -> one 6.43.8); ADR #34. Pin bumped, release-age exclude updated, new check-codemirror-dedupe.mjs CI guard, Playwright token-class assertion, stale dual-view comments cleaned. State left unsplit (6.7.1 is latest + satisfies view); only view overridden.

  2. Vendor the duckdb-wasm autocomplete extension (extends ADR #32): added to fetch + check scripts, manifest + 2 wasm binaries committed. Coordinator unchanged (lazy custom_extension_repository autoload).

  3. Drop the pull_request trigger from ci.yml (temporary, re-enable is a follow-up); push + concurrency intact, release.yml untouched.

  4. Rolling dev image tag on dev pushes (ADR #35 extends #21): dev push now publishes dev (moving) + dev- + sha-; main/v* unchanged; release.yml header comment updated in lockstep.

Also: sql-query.md flips the highlighting deferral to shipped (analyst); ARCHITECTURE/COMPONENTS/ci-cd-design living docs updated; v1.1.1.md Plan layer appended.

v1.1.2: scope opened to a second item — investigate bumping pnpm off 11.20.0 (minimumReleaseAge 'Invalid time value' crash). Framed as investigate-then-decide; preliminary finding: latest pnpm 11.21.0 changelog does NOT list a fix, so a bump is not evidence-based until reproduced.

Co-Authored-By: Xavier's assistant assistant@gijoe88.com

Four-item patch on the v1.1 line (architect design in docs/architecture/v1.1.1-design.md; ADRs #34 + #35 appended). 1. Restore SQL syntax highlighting — @codemirror/view de-duplicated to a single instance via a pnpm override (6.7.1 pin + 6.43.8 transitive -> one 6.43.8); ADR #34. Pin bumped, release-age exclude updated, new check-codemirror-dedupe.mjs CI guard, Playwright token-class assertion, stale dual-view comments cleaned. State left unsplit (6.7.1 is latest + satisfies view); only view overridden. 2. Vendor the duckdb-wasm autocomplete extension (extends ADR #32): added to fetch + check scripts, manifest + 2 wasm binaries committed. Coordinator unchanged (lazy custom_extension_repository autoload). 3. Drop the pull_request trigger from ci.yml (temporary, re-enable is a follow-up); push + concurrency intact, release.yml untouched. 4. Rolling dev image tag on dev pushes (ADR #35 extends #21): dev push now publishes dev (moving) + dev-<short-sha> + sha-<full>; main/v* unchanged; release.yml header comment updated in lockstep. Also: sql-query.md flips the highlighting deferral to shipped (analyst); ARCHITECTURE/COMPONENTS/ci-cd-design living docs updated; v1.1.1.md Plan layer appended. v1.1.2: scope opened to a second item — investigate bumping pnpm off 11.20.0 (minimumReleaseAge 'Invalid time value' crash). Framed as investigate-then-decide; preliminary finding: latest pnpm 11.21.0 changelog does NOT list a fix, so a bump is not evidence-based until reproduced. Co-Authored-By: Xavier's assistant <assistant@gijoe88.com>
feat(v1.1.1): SQL highlight fix, autocomplete ext, dev tag + CI PR-trigger
All checks were successful
CI / pnpm gate (typecheck, lint, build, test) (push) Successful in 1m35s
Details
release / build + scan + publish (push) Successful in 48s
Details
4019374341 
Four-item patch on the v1.1 line (architect design in docs/architecture/v1.1.1-design.md; ADRs #34 + #35 appended).

1. Restore SQL syntax highlighting — @codemirror/view de-duplicated to a single instance via a pnpm override (6.7.1 pin + 6.43.8 transitive -> one 6.43.8); ADR #34. Pin bumped, release-age exclude updated, new check-codemirror-dedupe.mjs CI guard, Playwright token-class assertion, stale dual-view comments cleaned. State left unsplit (6.7.1 is latest + satisfies view); only view overridden.

2. Vendor the duckdb-wasm autocomplete extension (extends ADR #32): added to fetch + check scripts, manifest + 2 wasm binaries committed. Coordinator unchanged (lazy custom_extension_repository autoload).

3. Drop the pull_request trigger from ci.yml (temporary, re-enable is a follow-up); push + concurrency intact, release.yml untouched.

4. Rolling dev image tag on dev pushes (ADR #35 extends #21): dev push now publishes dev (moving) + dev-<short-sha> + sha-<full>; main/v* unchanged; release.yml header comment updated in lockstep.

Also: sql-query.md flips the highlighting deferral to shipped (analyst); ARCHITECTURE/COMPONENTS/ci-cd-design living docs updated; v1.1.1.md Plan layer appended.

v1.1.2: scope opened to a second item — investigate bumping pnpm off 11.20.0 (minimumReleaseAge 'Invalid time value' crash). Framed as investigate-then-decide; preliminary finding: latest pnpm 11.21.0 changelog does NOT list a fix, so a bump is not evidence-based until reproduced.

Co-Authored-By: Xavier's assistant <assistant@gijoe88.com>
xavier merged commit 4019374341 into main 2026-08-14 09:46:36 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
xavier/s3-vedrfolnir!4
No description provided.