Skip to content

[codex] Remove stale cms notify references#709

Draft
lumberman wants to merge 1 commit intomainfrom
codex/remove-legacy-cms-notify
Draft

[codex] Remove stale cms notify references#709
lumberman wants to merge 1 commit intomainfrom
codex/remove-legacy-cms-notify

Conversation

@lumberman
Copy link
Copy Markdown
Collaborator

Summary

  • remove stale cms_notify and notifyCms manager runtime references
  • delete the unused legacy manager job form that still posted an unsupported notifyCms payload
  • close the follow-up review findings by fixing the job.ts provenance comment and reconciling the completed todo narrative

Why

The manager app still carried dead CMS notification vocabulary even though the live workflow and jobs API no longer used it. Removing those traces makes the manager UI and type surface more truthful and leaves cms_sync as a future plan-only concept until a real implementation exists.

Testing

  • pnpm --filter @forge/manager typecheck
  • pnpm --filter @forge/manager lint
  • pnpm --filter @forge/manager test
  • commit hooks also ran staged-file eslint and prettier

Post-Deploy Monitoring & Validation

  • No additional operational monitoring required: this PR removes unused manager code and updates documentation/todo artifacts without introducing a new runtime path.

Screenshots

  • No screenshots required: this change removes an unused legacy form component and updates internal copy/types rather than changing an active user-facing screen.

@railway-app
Copy link
Copy Markdown

railway-app Bot commented Apr 10, 2026

🚅 Deployed to the forge-pr-709 environment in forge

Service Status Web Updated (UTC)
@forge/manager ✅ Success (View Logs) Apr 10, 2026 at 11:47 am
@forge/cms ✅ Success (View Logs) Apr 10, 2026 at 11:47 am
1 service not affected by this PR
  • @forge/web

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