Skip to content

Persist Nova thread settings across navigation - #1388

Draft
ishaanxgupta wants to merge 1 commit into
mainfrom
codex/nova-thread-settings-persistence
Draft

Persist Nova thread settings across navigation#1388
ishaanxgupta wants to merge 1 commit into
mainfrom
codex/nova-thread-settings-persistence

Conversation

@ishaanxgupta

Copy link
Copy Markdown
Contributor

Summary

  • persist Nova model, reasoning, and space settings per thread
  • restore settings when reopening a conversation
  • keep Auto mode separate from the physical default space

Testing

  • bunx vitest run apps/web/lib/chat-thread-settings.test.ts
  • bunx biome check apps/web/components/chat/index.tsx apps/web/lib/chat-thread-settings.ts apps/web/lib/chat-thread-settings.test.ts
  • touched-file TypeScript check

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 31, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
supermemory-app 8226e60 Commit Preview URL

Branch Preview URL
Jul 31 2026, 02:11 PM

ishaanxgupta commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

How to use the Graphite Merge Queue

Add the label Main to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-mcp 8226e60 Jul 31 2026, 02:09 PM

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please revmoe this test file

typeof value === "object" && value !== null
? (value as Record<string, unknown>)
: {}
const model = isModelId(settings.model) ? settings.model : "grok-4.3"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets use grok-4.5 not 4.3

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.

2 participants