ENG-949: Add query sections to the left sidebar

[sid597]

https://www.loom.com/share/4981b1b58cfc4db1b70b766029858903

Summary by CodeRabbit

• New Features
  • Query sections can now be added to the personal sidebar for executing saved queries on-demand.
  • Query results display as clickable items in the sidebar with customizable result limits.
  • Added refresh option and dedicated menu for managing query sections in the sidebar.
  • Query sections support alias names and configurable result limits via settings.

[linear-code]

ENG-949 Add query and query results in the left sidebar

[supabase]

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.

---

Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

[sid597]

@coderabbitai full review

[coderabbitai]

✅ Actions performed

Full review triggered.

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR extends the left sidebar to support query sections—personal sections containing Roam query syntax in double parentheses. Users can execute queries on-demand, set result limits, and alias sections; query results are rendered as clickable child items with optional truncation.

Changes

Query Sections: Type Definitions, Settings Persistence, and Rendering

Layer / File(s)	Summary
Type Definitions & Helpers apps/roam/src/utils/getLeftSidebarSettings.ts	Added isQuerySection(text: string) to detect ((...)) syntax. Extended LeftSidebarPersonalSectionSettings with optional alias: AliasSetting and resultLimit: IntSetting fields.
Settings Parsing apps/roam/src/utils/getLeftSidebarSettings.ts	Updated getPersonalSectionSettings to parse "Alias" and "Result-limit" block nodes into settings shape, returning uid/value/valueUid structures.
Settings UI & Block Syncing apps/roam/src/components/settings/LeftSidebarPersonalSettings.tsx	Added inline InputGroup alias editor with debounced persistence via updateBlock/createBlock. Introduced PersonalNumberPanel for Result-limit with sync to block props. New section creation now generates Alias, Folded, Truncate-result?, and Result-limit backing blocks for query sections. Updated sectionsToBlockProps to write Alias and Result-limit props.
Query Execution & View Rendering apps/roam/src/components/LeftSidebarView.tsx	Added QuerySectionItem component that executes queries via fireQuery(parseQuery(queryUid)), manages loading/error/result states, applies resultLimit truncation, renders results as clickable child items, and provides popover menu for refresh and navigation. Integrated QuerySectionItem into PersonalSections rendering via isQuerySection detection.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• ENG-1256: Allow editing of left sidebar section names #677 — Modifies section-editing UI in LeftSidebarPersonalSettings; both PRs add persisted block-backed section metadata and update UI patterns.
• ENG-1217: Port section component in left sidebar #804 — Updates LeftSidebarPersonalSettings section initialization and block-prop syncing logic; overlaps with settings wiring in this PR.
• ENG-853 left sidebar global and personal settings #441 — Introduced the baseline left-sidebar personal settings infrastructure that this PR extends with query-section detection and query-specific settings handling.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: adding query sections to the left sidebar, which matches the primary objective and core changes across all three modified files.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Review rate limit: 0/1 reviews remaining, refill in 60 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 4

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@apps/roam/src/components/LeftSidebarView.tsx`:
- Around line 320-342: The runQuery function calls parseQuery(queryUid)
synchronously so any thrown parse errors bypass the promise .catch; update
runQuery to handle sync failures by either wrapping parseQuery in a try/catch
that sets setError("Query failed to run"), setIsLoading(false) and
setHasLoaded(true) on error, or convert to a promise chain (e.g.,
Promise.resolve().then(() => parseQuery(queryUid)).then(args =>
fireQuery(args)...)) so parse errors flow into the existing .catch; ensure you
reference runQuery, parseQuery, fireQuery, setError, setIsLoading and
setHasLoaded when applying the fix.

In `@apps/roam/src/components/settings/LeftSidebarPersonalSettings.tsx`:
- Line 944: Rename the setter callback parameter named `_keys` to a camelCase
identifier (e.g., `keys` or `selectedKeys`) in the setter prop passed in
LeftSidebarPersonalSettings (the arrow function `setter={(_keys, value) => { ...
}}`) to satisfy ESLint camelCase rules; update all references inside that
function body to the new name and ensure the function signature remains `(keys,
value) => { ... }` (or `(selectedKeys, value) => { ... }`) so no other behavior
changes.
- Around line 339-380: handleAliasChange is calling
syncAllSectionsToBlockProps(sectionsRef.current) while sections are not updated,
causing stale alias writes; fix by computing an updatedSections array (map over
previous sections and set the new alias.value and valueUid as appropriate) for
both the updateBlock and createBlock branches, immediately assign it to
sectionsRef.current and call setSections(updatedSections), then call
syncAllSectionsToBlockProps(updatedSections) (instead of sectionsRef.current)
before refreshAndNotify so the serialized settings reflect the new alias state;
reference handleAliasChange, alias.valueUid, updateBlock, createBlock,
setSections, sectionsRef, and syncAllSectionsToBlockProps.

In `@apps/roam/src/utils/getLeftSidebarSettings.ts`:
- Around line 22-23: isQuerySection currently treats any string wrapped in
((...)) as a query ref, allowing malformed refs like (()) or ((not-a-uid)) to
reach QuerySectionItem and parseQuery; update isQuerySection to extract the
inner text (e.g., const inner = text.slice(2, -2).trim()) and return true only
if inner is non-empty and passes a UID validation (prefer reusing an existing
isValidUid/isUid utility if available, otherwise validate against the Roam UID
pattern such as /^[A-Za-z0-9]{9}$/); this prevents invalid refs from being
classified as query sections and stops bad input from reaching
QuerySectionItem/parseQuery.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 9e3dab95-2fb3-4fea-9821-e85e93b2b71d

📥 Commits

Reviewing files that changed from the base of the PR and between fff0556 and 1de8531.

📒 Files selected for processing (3)

• apps/roam/src/components/LeftSidebarView.tsx
• apps/roam/src/components/settings/LeftSidebarPersonalSettings.tsx
• apps/roam/src/utils/getLeftSidebarSettings.ts