Skip to content

DO NOT MERGE: test publish script#15

Closed
argshook wants to merge 1 commit into
mainfrom
test-should-not-publish
Closed

DO NOT MERGE: test publish script#15
argshook wants to merge 1 commit into
mainfrom
test-should-not-publish

Conversation

@argshook

Copy link
Copy Markdown
Contributor

No description provided.

@argshook argshook self-assigned this Jun 23, 2022
@argshook

Copy link
Copy Markdown
Contributor Author

cool, tested, thanks!

@argshook argshook closed this Jun 23, 2022
@argshook argshook deleted the test-should-not-publish branch June 23, 2022 13:04
emrberk added a commit that referenced this pull request Jul 14, 2026
Address review findings on the agent-passive-edit path:

- #8 archived preview: seed a read-only preview from the persisted view
  instead of erroring; NotebookProvider gains a `preview` prop that skips
  live-controller registration and persistence; key <Notebook> on archived
  so restore remounts cleanly. Also fixes the restore-from-search stale
  archived flag this surfaced.
- #6 archive/commit race: commitView re-reads the row inside a Dexie
  transaction and returns committed|archived|deleted, rejecting archived
  and deleted rows atomically with the write.
- #3 reconnect freshness: per-generation guard so an in-flight read that
  spans a reconnect reset no longer seeds freshness for the new connection.
- #7 apply_notebook_state: report state_applied + post_apply_aborted on a
  post-commit abort instead of a generic failure.
- #9 hide-result: bump the freshness seq so a stale agent read is caught.
- #15 agent-change toast a11y: persistent live announcer, return focus to
  the pill, split hover/focus auto-hide pause.
- Surface the active tab's archived state to the agent (workspace + digest).

Tests: freshness generation guard, commitView archived/deleted, and the
apply_notebook_state post-commit abort path.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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