Skip to content
Puneet Moger

Open source

In other codebases.

Small fixes in projects I use. From database adapters to frontend types, here’s what I changed and how I checked it.

6 merged1 open PR4 projects

Select a contribution to see the PR details

Kestra / #21610

Mergedkestra-io/kestra

fix(executions): type restart and breakpoint resume actions

Execution restart and breakpoint resume actions used explicit any types, weakening type checking in Kestra’s frontend.

Opened
Oct 21, 2026
Merged
Oct 22, 2026
Files
3 changed
Changes
+14 / −18

What I changed

  • Use FlowForExecution for the loaded flow.
  • Call typed restart and replay store methods directly.
  • Type the breakpoint resume prop as Execution and regenerate the explicit-any baseline.

Validation reported in the PR

The PR reports passing type checks, 3,946 unit tests, two breakpoint-resume Storybook tests, lint, and the explicit-any baseline check. The production build was not run.

View PR on GitHub

LibreDB / #1911

Mergedlibredb/libredb-studio

fix: avoid estimate mismatch warnings below Limit

An ancestor LIMIT can stop a query plan node early, causing a misleading estimate mismatch warning even when the query behaves correctly.

Opened
Oct 21, 2026
Merged
Oct 21, 2026
Files
3 changed
Changes
+179 / −4

What I changed

  • Track ancestor LIMIT context and suppress only downward mismatches for nodes that can stop early.
  • Reset that context below blocking operators such as Sort, Hash, and non-streaming aggregates.
  • Add regression coverage for nested limits, streaming operators, and blocking operators.

Validation reported in the PR

The PR reports 77 VisualExplain tests passing, 11 warning scenarios checked in Chrome, and successful SQLite execution and Explain checks. The full repository suite and coverage gate were not verified.

View PR on GitHub

ToolJet / #18350

Open for reviewToolJet/ToolJet

fix(cohere): load models live from Cohere API, add Embed and Rerank

ToolJet’s Cohere plugin used a stale, hardcoded model list. Users could not select current models, and retired models caused requests to fail.

Opened
Oct 9, 2026
Status
Awaiting review
Files
10 changed
Changes
+385 / −670

What I changed

  • Fetch the available models directly from Cohere for chat, embed, and rerank.
  • Add Embed and Rerank operations and remove the hardcoded model from connection testing.
  • Replace duplicated model fields with shared schema fields, update the docs, and preserve saved query model values.

Validation reported in the PR

The PR reports passing marketplace builds, plugin validation, and lint. The live Cohere Models API and an Embed query were also checked. This PR is still open for review.

View PR on GitHub

LibreDB / #1805

Mergedlibredb/libredb-studio

fix(mssql): report the server's 4060 when the database does not exist…

A database name that did not exist was reported as a login failure, making a missing database indistinguishable from incorrect credentials.

Opened
Oct 16, 2026
Merged
Oct 19, 2026
Files
5 changed
Changes
+103 / −7

What I changed

  • Capture SQL Server error 4060 from the connection’s error event.
  • Classify missing-database errors before authentication failures.
  • Update the MSSQL provider documentation and add unit and integration regression tests.

Validation reported in the PR

The PR reports checking the mocked driver’s error ordering. The full coverage run did not complete locally, so the coverage gate was not verified.

View PR on GitHub

Data Peek / #330

MergedRohithgilla12/data-peek

fix(mssql): list sequences from sys.sequences instead of returning []

The SQL Server adapter returned an empty sequence list even though SQL Server has supported sequences since 2012.

Opened
Oct 8, 2026
Merged
Oct 9, 2026
Files
2 changed
Changes
+25 / −5

What I changed

  • Query sys.sequences with schema and type information, excluding system schemas.
  • Map results to the existing SequenceInfo contract.
  • Correct the adapter documentation to describe databases with and without sequence support.

Validation reported in the PR

The PR reports passing Node type checks and 1,486 desktop tests. The query was not tested against a live SQL Server. Showing sequences in the schema explorer is separate follow-up work.

View PR on GitHub

Data Peek / #301

MergedRohithgilla12/data-peek

fix(desktop): show empty states instead of raw errors on the SQLite Health Monitor

SQLite connections showed raw adapter errors in Health Monitor panels that only apply to server databases.

Opened
Oct 5, 2026
Merged
Oct 6, 2026
Files
2 changed
Changes
+94 / −51

What I changed

  • Identify embedded SQLite connections and show explanatory empty states.
  • Suppress loading indicators for unsupported health panels.
  • Keep the working Table Sizes panel and the Health Monitor sidebar entry available.

Validation reported in the PR

The PR reports 1,425 desktop tests passing and a clean lint check for the changed component. The SQLite empty states were checked visually.

View PR on GitHub

LibreDB / #1465

Mergedlibredb/libredb-studio

test: remove stale engine counts from LoginPage test comments

Comments in the LoginPage tests contained hardcoded engine counts that became outdated as database support expanded.

Opened
Oct 4, 2026
Merged
Oct 4, 2026
Files
1 changed
Changes
+6 / −5

What I changed

  • Replace three stale counts with descriptions of the relationships being tested.
  • Keep the assertions and runtime behavior unchanged.

Validation reported in the PR

The PR records a self-review of this comment-only change. Tests were not run for this PR.

View PR on GitHub

PR statuses checked on 22 October 2026.