Skip to content

Fix/kiloclaw morning briefing graceful unavailable#3082

Merged
St0rmz1 merged 3 commits intomainfrom
fix/kiloclaw-morning-briefing-graceful-unavailable
May 6, 2026
Merged

Fix/kiloclaw morning briefing graceful unavailable#3082
St0rmz1 merged 3 commits intomainfrom
fix/kiloclaw-morning-briefing-graceful-unavailable

Conversation

@St0rmz1
Copy link
Copy Markdown
Contributor

@St0rmz1 St0rmz1 commented May 6, 2026

Summary

When a KiloClaw instance runs an image that predates the Morning Briefing status route, the worker now responds with HTTP 200 and a typed payload that includes code: controller_route_unavailable, instead of returning HTTP 404. The web UI reads this code on the Settings tab and renders an upgrade affordance inside the Morning Briefing card. The card disables the Enable, Disable, and Generate controls while the user is on an older image and routes the upgrade button into the upgrade modal that already exists for the Memory section. Warmup state is suppressed when the upgrade banner is shown so the two states do not compete.

Verification

  • Personal /claw/settings on a current image: Morning Briefing card renders as before with status, schedule, last delivery, and working controls.
  • Personal /claw/settings with the worker stubbed to return code: controller_route_unavailable: amber Upgrade Required banner shows inside the card, badge reads Upgrade Required, schedule and last delivery rows are hidden, controls are disabled, clicking Upgrade opens the upgrade modal.
  • Organization /organizations/[id]/claw/settings: same two cases as above.

Visual Changes

Before After

Reviewer Notes

The new payload mirrors the gateway warming up branch in the same worker handler, so the frontend can key on briefingStatus.code without tRPC changes.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 6, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (5 files)
  • apps/web/src/app/(app)/claw/components/SettingsTab.tsx
  • apps/web/src/app/(app)/claw/components/morning-briefing-card-state.ts
  • apps/web/src/app/(app)/claw/components/morning-briefing-card-state.test.ts
  • services/kiloclaw/src/routes/platform.ts
  • services/kiloclaw/src/routes/platform-morning-briefing.test.ts

Reviewed by gpt-5.5-20260423 · 1,447,685 tokens

@St0rmz1 St0rmz1 merged commit 7235bb2 into main May 6, 2026
13 checks passed
@St0rmz1 St0rmz1 deleted the fix/kiloclaw-morning-briefing-graceful-unavailable branch May 6, 2026 21:16
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.

2 participants