janpeter/Ops-dashboard
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[codex] Fix Ops idea update PATCH route #89

Merged
janpeter merged 1 commit from codex/fix-ops-idea-patch-update into main 2026-06-14 21:03:44 +02:00
Conversation 0 Commits 1 Files changed 3 +6 -2
janpeter commented 2026-06-14 20:59:50 +02:00
Owner
Copy link

Summary

  • Export PATCH from the Ops-dashboard /api/s4m/[...path] catch-all route so idea updates reach the copilot kit/service.
  • Add a regression guard that verifies the host route keeps exporting PATCH.
  • Update the vendored scrum4me-copilot submodule to the version containing the ideas CRUD + PATCH route support.

Root cause

Adding an idea used POST, but updating an idea uses PATCH. Ops-dashboard only exported GET and POST from createS4MRoutes, and its vendored copilot submodule was still on a version that did not expose the idea update route/type surface.

Validation

  • npm test passed: 13 test files, 80 tests.
  • npm run typecheck still fails on an unrelated existing issue: lib/ingest-worker-log.ts:207 uses pool, which is not present on RunRow.

Notes

  • Local lib/build-info.ts changes were intentionally left out of this branch.
## Summary - Export `PATCH` from the Ops-dashboard `/api/s4m/[...path]` catch-all route so idea updates reach the copilot kit/service. - Add a regression guard that verifies the host route keeps exporting `PATCH`. - Update the vendored `scrum4me-copilot` submodule to the version containing the ideas CRUD + PATCH route support. ## Root cause Adding an idea used `POST`, but updating an idea uses `PATCH`. Ops-dashboard only exported `GET` and `POST` from `createS4MRoutes`, and its vendored copilot submodule was still on a version that did not expose the idea update route/type surface. ## Validation - `npm test` passed: 13 test files, 80 tests. - `npm run typecheck` still fails on an unrelated existing issue: `lib/ingest-worker-log.ts:207` uses `pool`, which is not present on `RunRow`. ## Notes - Local `lib/build-info.ts` changes were intentionally left out of this branch.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/Ops-dashboard!89
No description provided.