Skip to content

feat(connections): offer Quack in Add connection - #79

Merged
caioricciuti merged 1 commit into
mainfrom
feat/quack-in-add-connection
Oct 23, 2026
Merged

caioricciuti merged 1 commit into
mainfrom
feat/quack-in-add-connection

Conversation

@caioricciuti

Copy link
Copy Markdown
Owner

"Add connection" is where people look first, and Quack was not there: it had its own Attach button in the Remote DuckDB panel. The Connection Type list now offers "Remote DuckDB (Quack)", which closes the sheet and opens the attach sheet. One attach form, two ways in.

  • QuackAttachments takes a bindable attaching prop, so the page can open its sheet; fields reset each time it opens.
  • Docs: the attach step names both ways.
  • e2e: picking Quack hands over to the attach sheet, and Add connection reopens with its own defaults. No server needed, runs on CI.
  • quack.spec.ts now skips unless the duckdb CLI can load the quack extension, not just when the CLI is missing. Homebrew's 2.0.0 formula ships a binary that reports a dev build and finds no published extensions, which made the spec fail in beforeAll.

No dependency changes.

Verified

lint, format, typecheck, 1213 unit tests, build (index 306.81 kB), site build. Locally the e2e suite had 38 passed, 2 skipped (Quack, see above) and 6 failed: all liveSession.spec.ts, peers never connect. The same test fails on main in a clean worktree, so it is not this change; this CI run will show whether it is local to that machine.

@pkgfire

pkgfire Bot commented Oct 23, 2026

Copy link
Copy Markdown

PkgFire: WARN · scan details

No dependency changed. 723 packages checked.

Why

  • WARN intel_stale kev : The kev exploitation feed last refreshed 2 days ago, past the 48-hour limit.
Listed, not counted
  • new_publisher: 10 packages
  • established_version: 126 packages
  • provenance_lost: 4 packages
  • already_on_base: 22 packages
  • popular_name: 2 packages

@caioricciuti
caioricciuti merged commit ac68b7b into main Oct 23, 2026
6 checks passed
@caioricciuti
caioricciuti deleted the feat/quack-in-add-connection branch October 23, 2026 08:50
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