CS-408 [Bug] - org chart image doesn't update after upload, delete, or replace

[github-actions]

This is an automated pull request to merge chas/org-chart-update into dev.
It was created by the [Auto Pull Request] action.

---

Summary by cubic

Make the org chart update instantly after upload, replace, delete, or editor save by calling useOrgChart().mutate via an onChartChange callback. Fixes stale data in the org chart view and removes router.refresh() (CS-408).

• Bug Fixes
  • Thread onChartChange through OrgChartContent, OrgChartEmptyState, OrgChartImageView, OrgChartEditor, and UploadOrgChartDialog.
  • Trigger onChartChange after upload, delete, replace, and save; wrap calls in try/catch so SWR refresh failures don’t affect user actions.
  • Wire UploadOrgChartDialog onUploaded to useOrgChart().mutate via OrgChartTabContent.

^{Written for commit 7f733c6. Summary will update on new commits.}

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
app	Ready	Preview, Comment	Jul 7, 2026 3:46pm
comp-framework-editor	Ready	Preview, Comment	Jul 7, 2026 3:46pm

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
portal	Skipped		Jul 7, 2026 3:46pm

[linear]

CS-408

[cubic-dev-ai]

3 issues found across 5 files

Confidence score: 3/5

• In apps/app/src/app/(app)/[orgId]/people/org-chart/components/OrgChartContent.tsx, onChartChange (SWR mutate) is not passed to OrgChartEditor, so editor saves may not refresh parent cache and users can see stale org chart data after successful edits — wire onChartChange through the editor save path before merging.
• In apps/app/src/app/(app)/[orgId]/people/org-chart/components/UploadOrgChartDialog.tsx, await onUploaded() is inside the same try as the upload call, so a successful upload can still surface a failure toast if refresh fails, which is confusing and may trigger duplicate retries — separate upload success handling from refresh error handling (or use distinct messaging).
• In apps/app/src/app/(app)/[orgId]/people/org-chart/components/OrgChartImageView.tsx, a thrown onChartChange() after a successful delete is reported as “Failed to delete org chart,” masking that the delete already completed — split delete and re-fetch error paths so users get accurate feedback and next steps.

<sub>Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic</sub>

[chasprowebdev]

@cubic-dev-ai please review it.

[cubic-dev-ai]

@cubic-dev-ai please review it.

@chasprowebdev I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 6 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Re-trigger cubic}