refactor: rename dashboard references to workspace terminology

[urjitc]

This PR renames all dashboard-related components to workspace terminology and removes legacy /dashboard routes entirely.

• Move DashboardLayout to WorkspaceLayout in src/components/layout
• Extract shell logic from src/app/dashboard/page.tsx to src/components/workspace/WorkspaceShell.tsx
• Update src/app/workspace/[slug]/page.tsx to import from new component location
• Delete src/app/dashboard/ directory (legacy redirect routes)
• Remove /dashboard/ pathname fallback in WorkspaceContext.tsx
• Remove /dashboard/ from robots.txt disallow list
• Remove dashboard keyword from home/layout.tsx SEO metadata
• Update component references in comments across 6 files

Summary by CodeRabbit

• Refactor

  • Removed legacy /dashboard/[slug] route and deprecated client-side redirects; routing now uses /workspace/ paths.
  • Renamed dashboard components, layouts, and documentation to workspace equivalents.

• Other

  • robots.txt no longer disallows /dashboard.
  • Home metadata keywords updated (removed "dashboard").
  • Minor comment/documentation updates and modal redirect comment adjusted.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
thinkex	Ready	Preview, Comment	Apr 29, 2026 9:26pm

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d385fb9d-7b98-42c7-90d3-bf8c10a21b92

📥 Commits

Reviewing files that changed from the base of the PR and between 5b5b6ab and 1246f73.

📒 Files selected for processing (1)

• src/components/workspace/WorkspaceShell.tsx

---

📝 Walkthrough

Walkthrough

Removes the legacy /dashboard/[slug] client redirect and drops the /dashboard/ disallow from robots.txt; renames dashboard-related components, exports, and comments to workspace variants; updates Home metadata keywords; and stops deriving currentSlug from /dashboard paths in WorkspaceContext.

Changes

Cohort / File(s)	Summary
Routing & robots public/robots.txt, src/app/dashboard/[slug]/page.tsx	Removes the robots.txt disallow for /dashboard/ and deletes the client redirect page that forwarded /dashboard/[slug] → /workspace/[slug].
Workspace shell & page renames src/components/workspace/WorkspaceShell.tsx, src/app/workspace/[slug]/page.tsx	Renames Dashboard* exports to Workspace* (Page, Shell, Content, View); Workspace page now renders WorkspaceShell and removed a default client redirect Page.
Layout component rename src/components/layout/WorkspaceLayout.tsx	Renames DashboardLayout and its props → WorkspaceLayout/WorkspaceLayoutProps (JSDoc and export rename only).
Context slug derivation src/contexts/WorkspaceContext.tsx	Removes logic that derived currentSlug from /dashboard/... paths so only /workspace/<slug> returns a non-null slug.
Metadata and minor comments src/app/home/layout.tsx, src/app/share-copy/[id]/page.tsx, src/components/chat/ChatPanel.tsx, src/components/modals/PasswordProtectedPdfDialog.tsx, src/components/workspace-canvas/WorkspaceSection.tsx, src/lib/layout-constants.ts, src/lib/stores/ui-store.ts	Removed "dashboard" keyword from home metadata and updated inline/JSDoc comments from “dashboard” → “workspace”; no runtime behavior changes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• feat: use tanstack query for workspaces #148 — touches src/contexts/WorkspaceContext.tsx and slug derivation behavior.
• Feature/issue 32/home route clarity #33 — overlaps on removal/rerouting of /dashboard/[slug] and routing consolidation.
• Feat/optimize workspace loading #149 — modifies workspace-loading/context logic that interacts with slug/loading behavior.

Suggested labels

enhancement

Poem

🐰 I hopped from Dashboard to Workspace light,
Renamed the shells and set slugs right.
Old routes folded, comments aligned,
Code tidied up — a tidy mind.
Hooray for order, carrot-bright! 🥕

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 70.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'refactor: rename dashboard references to workspace terminology' accurately and specifically summarizes the main change—a comprehensive renaming of dashboard components and references to workspace terminology throughout the codebase, including removal of legacy routes.
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.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch capy/dashboard-to-workspace

---

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.}

[greptile-apps]

Greptile Summary

This PR renames all Dashboard* components and routes to Workspace* terminology, extracts WorkspaceShell from the legacy src/app/dashboard/page.tsx into src/components/workspace/WorkspaceShell.tsx, and hard-deletes the /dashboard and /dashboard/[slug] app routes.

• The deleted routes had client-side redirects to /workspace and /workspace/[slug], but no equivalent server-side redirects were added to next.config.ts — users with bookmarked /dashboard/… URLs will receive 404s after this lands.

Confidence Score: 3/5

Not safe to merge without adding Next.js redirects for removed /dashboard routes; existing bookmarks and any indexed links will 404.

A P1 finding is present: the client-side redirects for /dashboard and /dashboard/[slug] are deleted with no server-side replacement, breaking backward compatibility for existing users. The rename logic itself is clean and complete.

next.config.ts needs permanent redirects added for /dashboard → /workspace and /dashboard/:slug → /workspace/:slug.

Important Files Changed

Filename	Overview
src/components/workspace/WorkspaceShell.tsx	Renamed from src/app/dashboard/page.tsx; DashboardShell → WorkspaceShell, internal components renamed, useRouter/default export removed. Minor: triple duplicate comment at lines 371-373.
next.config.ts	Not changed in the PR but is the correct location for permanent redirects that are missing — users who bookmarked /dashboard or /dashboard/[slug] will now hit 404.
src/app/dashboard/[slug]/page.tsx	Deleted legacy client-side redirect page for /dashboard/[slug] → /workspace/[slug]; deletion is clean but no server-side redirect replaces it.
src/contexts/WorkspaceContext.tsx	Removed /dashboard/ pathname fallback from currentSlug derivation; straightforward cleanup with no side-effects.
src/components/layout/WorkspaceLayout.tsx	Renamed from DashboardLayout; interface and function name updated, logic unchanged.
src/app/workspace/[slug]/page.tsx	Updated import from DashboardShell (old app route) to WorkspaceShell (new component path); correct and complete.
public/robots.txt	Removed Disallow: /dashboard/ entry since the route no longer exists; correct cleanup.
src/app/home/layout.tsx	Removed "dashboard" from SEO keywords array; trivial metadata cleanup.

Comments Outside Diff (1)

1. next.config.ts, line 5-36 (link)

   Missing permanent redirects for removed /dashboard routes

   The PR deletes both the /dashboard (root, redirected to /workspace) and /dashboard/[slug] (redirected to /workspace/[slug]) app routes. With no Next.js-level redirects() configured, any user who bookmarked a /dashboard/… URL — or any crawled link still pointing there — will now receive a 404 instead of being transparently forwarded. Consider adding permanent redirects to next.config.ts:

   async redirects() {
     return [
       { source: "/dashboard", destination: "/workspace", permanent: true },
       { source: "/dashboard/:slug", destination: "/workspace/:slug", permanent: true },
     ];
   },

   

_{Reviews (1): Last reviewed commit: "Rename dashboard to workspace terminolog..." | Re-trigger Greptile}

[greptile-apps]

Duplicate stale comment block

Lines 371–372 are identical left-over duplicates of the original // Main page component comment — the rename pass introduced a third copy instead of collapsing them. Only one comment (the descriptive (wrapper) variant) is needed here.

Suggested change

	// Main page component
	// Main page component
	// Main page component (wrapper)
	// Main page component (wrapper)
	export function WorkspacePage() {

[cubic-dev-ai]

No issues found across 13 files

_{Requires human review: This is a significant terminology refactor that removes legacy routes and renames core layout components. It has a high blast radius and modifies routing logic.}