feat: notebooks with coding agents integration#561
Draft
emrberk wants to merge 53 commits into
Draft
Conversation
emrberk
changed the title
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
…nge stripping logic
…-only pairing default - key result snapshots to the executed SQL and guard the single-query completion write so mid-run edits / agent invalidation can't mislabel or resurrect results; supersede run generations on unmount so aborted runs don't overwrite the last good snapshot - debounce the draw-cell fetch pipeline (300ms) so typing no longer fires validate/execute requests per keystroke - validate and sanitize notebookViewState on tab import (malformed imports crashed the notebook scene) - emit user_moved_cell from arrow-key reorder so paired agents see it - default new MCP pairings to read-only; write needs explicit upgrade - "Reset to auto" reuses the canonical first-draw chart inference Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…, designated timestamp highlight from exec response Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…duplicate lost on reload) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
A run cell tracks its outcome only in the live result during a session. buildAppliedCells dropped that result on any value change, wiping the run-history signal, so the next apply saw the cell as never-run and auto-executed the write again (duplicate INSERT/UPDATE/DROP). Collapse the dropped result into lastRunStatus so ranBefore stays true. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ve_value in apply_notebook_state Duplicated cells/notebooks keep their collapsed run status so apply's auto-run never re-fires a write that already ran through the original. Cell reads no longer truncate silently: get_cell caps at 4 KB with out-of-band truncated/full_length flags and takes get_full_content for the verbatim value (hard 1 MB ceiling, fails loudly); snapshot previews flag preview_truncated/full_length when cut. apply_notebook_state cells take exactly one of value / preserve_value:true, so agents keep cells they aren't changing instead of echoing possibly-stale or truncated copies; preserved cells retain value, results, and run history. STATE_STALE recovery text now steers agents to preserve_value and full re-reads so user edits survive the documented retry path. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
apply_notebook_state and add_cell(run:true) now skip cells containing DDL/DML regardless of run history or the write permission, returning skipped:true with a note pointing at an explicit, user-consented run_cell. Removes the fragile ranBefore distinction entirely. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ender - Drop the source id on validation-failed tabs during import so bulkAdd assigns a fresh key; keeping it could collide with an existing buffer and abort the whole import transaction, leaving nothing imported. - Include areaStyle/step/stack presence in the chart structural key so a subtype switch (e.g. area->line, stacked->grouped bar) forces a remount instead of an echarts merge that kept the previous fill/step/stack. Co-Authored-By: Claude Opus 4.8 (1M context) <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.
yarn && yarn startand see web console on localhost:9999 as before.yarn && yarn build. We will use the script ...clone path.../mcp-bridge/dist/index.jsin the next step.codex mcp add questdb --env CONSOLE_ORIGIN=http://localhost:9999/ -- ...clone path.../mcp-bridge/dist/index.js~/.claude.jsonon Mac). Then add this to the config:/mcp.