Add Nova workspace prompt settings#1323
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | 562e721 | Jul 21 2026, 04:36 PM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | 562e721 | Commit Preview URL Branch Preview URL |
Jul 21 2026, 04:37 PM |
| if (response.error) { | ||
| throw new Error(response.error.message || "Failed to save settings") |
There was a problem hiding this comment.
The style guide requires always using new Error() when throwing errors and including any supporting data as the cause argument instead of inlining it into the string. The error message is constructed by concatenating response.error.message with a fallback string (response.error.message || "Failed to save settings"). The supporting data (response.error.message) should be passed as the cause option: throw new Error("Failed to save settings", { cause: response.error.message }).
| if (response.error) { | |
| throw new Error(response.error.message || "Failed to save settings") | |
| if (response.error) { | |
| throw new Error("Failed to save settings", { cause: response.error.message }) | |
Spotted by Graphite (based on custom rule: TypeScript style guide (Google))
Is this helpful? React 👍 or 👎 to let us know.
There was a problem hiding this comment.
Addressed in 1ea6e2dd: the save failure now uses a stable Error message and preserves the API detail in cause. Focused Biome and git diff --check pass.
Adds a dedicated Workspace Prompt editor for Company Brain organizations while preserving the existing Organization Context ingestion-filter controls for every organization manager.
Changes
workspacePromptsupport to shared request, GET response, and PATCH response contracts.{ orgId, orgSlug, updated }API response.updatedsettings into the submitting organization’s cache, then exactly refetches that organization.filterPromptcompatibility, andX-App-Source: nova.Testing
packages/libandpackages/validationTypeScript checks.Session Details
(aside)to your comment to have me ignore it.