Skip to content

Render organization-managed dashboards in Desktop - #4082

Merged
reachjalil merged 1 commit into
devfrom
feature/desktop-managed-dashboards
Aug 25, 2026
Merged

reachjalil merged 1 commit into
devfrom
feature/desktop-managed-dashboards

Conversation

@reachjalil

Copy link
Copy Markdown
Member

Follow-up to #4056 after the managed-dashboard API became available.

Summary

  • fetch dashboards granted to the signed-in member after Den advertises the capability
  • render organization-managed dashboards and MCP App tiles in Desktop
  • keep dashboard authoring and MCP App selection server-side by removing the Desktop Add app flow and local dashboard store
  • preserve per-user, per-organization launch consent; managed tiles are non-removable, manual-first, and write-capable tools remain request-only

Verification

  • pnpm --dir apps/app typecheck
  • pnpm --dir apps/app exec bun test --isolate tests/den-managed-dashboards-capability.test.ts tests/granted-dashboard-store.test.ts — 5 passed
  • OPENWORK_EVAL_E2E_TESTS=1 pnpm --dir evals exec vitest run --config vitest.config.ts --project e2e specs/org-managed-dashboard-desktop.e2e.test.ts — 1 passed on the exact PR head

@vercel

vercel Bot commented Aug 25, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-app Ready Ready Preview Aug 25, 2026 6:28pm
openwork-den Ready Ready Preview Aug 25, 2026 6:28pm
openwork-den-worker-proxy Ready Ready Preview Aug 25, 2026 6:28pm
openwork-diagnostics Ready Ready Preview Aug 25, 2026 6:28pm
openwork-landing Ready Ready Preview, v0 Aug 25, 2026 6:28pm

@reachjalil

Copy link
Copy Markdown
Member Author

Test evidence — desktop-consumes-server-managed-dashboards-without-local-authoring-or-launch-con — 0/0 screenshots passed · 2 assertions

✅ 0/0 screenshots passed · 2 assertions · 2 expectations passed · 0 failed · 0 pending

screenshots not uploaded (no BLOB_READ_WRITE_TOKEN)

ℹ️ ASSERTION — 1. Desktop renders only dashboards granted to the signed-in member

org=org_01m0x2bpeve8yakzsam93n4qmr; granted=Operations board 1787681954317; ungranted=Private board 1787681954317; state={"boardVisible":true,"privateBoardVisible":false,"runVisible":true,"removeVisible":false,"addAppVisible":false,"managedLabelVisible":true}

  • ✅ PASS Desktop renders only dashboards granted to the signed-in member — org=org_01m0x2bpeve8yakzsam93n4qmr; granted=Operations board 1787681954317; ungranted=Private board 1787681954317; state={"boardVisible":true,"privateBoardVisible":false,"runVisible":true,"removeVisible":false,"addAppVisible":false,"managedLabelVisible":true}

ℹ️ ASSERTION — 2. Desktop has no local dashboard authoring, while managed tiles remain manual-first and non-removable

tile=Weekly report; state={"boardVisible":true,"privateBoardVisible":false,"runVisible":true,"removeVisible":false,"addAppVisible":false,"managedLabelVisible":true}

  • ✅ PASS Desktop has no local dashboard authoring, while managed tiles remain manual-first and non-removable — tile=Weekly report; state={"boardVisible":true,"privateBoardVisible":false,"runVisible":true,"removeVisible":false,"addAppVisible":false,"managedLabelVisible":true}

Test run created 2026-08-25T18:18:47.220Z · Source: evals/results/test-runs/2026-08-25T18-18-47-219Z-desktop-consumes-server-managed-dashboards-without-local-authoring-or-launch-con/test-run.json · Repro: pnpm --dir evals artifacts:publish -- --pr 4082 --test-run 2026-08-25T18-18-47-219Z-desktop-consumes-server-managed-dashboards-without-local-authoring-or-launch-con

@reachjalil
reachjalil enabled auto-merge (squash) August 25, 2026 18:29

@diff-warden diff-warden Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Warden security clearance: clear. No blocking findings (security or desktop↔den sync) in this diff (b51ca032e28e08c41b1e9c227f2ccbcc58686d04). Automated clearance satisfies the required-review gate only — a human still reviews and merges. Analysis run

@reachjalil
reachjalil merged commit 4921a02 into dev Aug 25, 2026
35 checks passed

This branch was successfully deployed

5 active deployments
Preview – openwork-den-worker-proxy — b51ca032 Deployed Aug 25, 2026 by vercel[bot]
Preview – openwork-landing — b51ca032 Deployed Aug 25, 2026 by vercel[bot]
Preview – openwork-den — b51ca032 Deployed Aug 25, 2026 by vercel[bot]
Preview – openwork-app — b51ca032 Deployed Aug 25, 2026 by vercel[bot]
Preview – openwork-diagnostics — b51ca032 Deployed Aug 25, 2026 by vercel[bot]
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