Include .nvmrc hash in the test pipeline cache key so a Node.js
version bump invalidates the cache. Add node_modules caching to
studio-publish and studio-build-non-production pipelines so
subsequent builds with the same deps skip yarn install. These
use runner.arch in the key since they build across x64 and arm64.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When a test or beforeEach fails before electronApp is assigned,
the afterEach would crash trying to call .close() on undefined,
masking the real error. Add a null check in all 9 e2e test files.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace direct Trino HTTPS configuration with nginx reverse proxy
for TLS termination, following Trino docs best practices. Add
http-server.process-forwarded=true to Trino config. Remove alpha
warning from TrinoForm.vue.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- buildSchemaFilter now uses escapeString() to escape single quotes in
schema, only, and ignore filter values.
- Trino driver now wraps this.db with wrapIdentifier() in all queries
(listSchemas, listTables, listTableColumns, buildPaginatedQuery).
- Trino listTableColumns escapes table and schema string literals.
- Added unit tests for SQL injection in buildSchemaFilter and Trino.
- listTables(null) no longer returns []. Uses buildSchemaFilter() to
optionally filter by schema, matching the PostgreSQL driver pattern.
- SSL/TLS connections now work. Certificate files (CA, cert, key) are
read and passed to trino-client. When no certs are provided,
rejectUnauthorized defaults to false (trust server cert).
- Removed server:ssl from Trino disabledFeatures so the SSL toggle
appears in the connection form.
- URL parser now auto-enables ssl for https:// URLs.
- Added integration tests for listTables and SSL (with HTTPS-enabled
Trino testcontainer).
- Added unit tests for SSL config propagation.
- Updated Trino connection docs (EN + ES) with SSL section.
Add a prep job that runs yarn install once and caches the full
node_modules tree via actions/cache. All downstream jobs (unit,
integration, e2e) restore from cache instead of running their own
yarn install, saving ~2-5 min per job across 20+ parallel jobs.
Also updates actions/checkout v2 -> v4, fixes deprecated set-output
syntax, and standardizes all jobs on ubuntu-24.04.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Cloud connections/queries must always be in a folder. The option
was silently moving items to the owner's personal folder when null
folder_id was submitted. Hidden in the UI; backend now rejects null
folder_id for workspace records.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Clear connectionFolderId when importing connections from local workspace
to prevent them from being placed in team folders due to coincidental
folder ID matches.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add connectionFolderId/queryFolderId to ReorderResult interface
- Update cloud modules to apply folder_id from reorder response
- Update test mocks to include folder_id
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add dedicated reorder API endpoint that returns all affected siblings
- Fix position collisions by updating all sibling positions from API response
- Unify local and cloud workspace reorder actions with same interface
- Fix local workspace bug where unsorted state caused incorrect positions
- Add pendingSaveIds tracking to prevent poll from overwriting optimistic updates
- Add comprehensive unit tests for reorder scenarios
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Connection and query folder open/closed state is now saved to localStorage
and restored on page load. New folders default to expanded. Uses separate
keys ('connectionFolderExpanded-v1' / 'queryFolderExpanded-v1') so the two
sidebar lists are independent.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
