Skip to content

Source diff message query pattern#26638

Merged
kitlangton merged 1 commit into
devfrom
cleanup/openapi-diff-message-query
May 10, 2026
Merged

Source diff message query pattern#26638
kitlangton merged 1 commit into
devfrom
cleanup/openapi-diff-message-query

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

@kitlangton kitlangton commented May 10, 2026

Summary

  • Remove the GET /session/{sessionID}/diff messageID query schema override from public.ts.
  • Cover that the query parameter pattern now comes from the runtime MessageID schema.

Testing

  • bun test --timeout 5000 test/server/httpapi-query-schema-drift.test.ts
  • bun typecheck from packages/opencode
  • bunx prettier --check packages/opencode/src/server/routes/instance/httpapi/public.ts packages/opencode/test/server/httpapi-query-schema-drift.test.ts
  • bunx oxlint packages/opencode/src/server/routes/instance/httpapi/public.ts packages/opencode/test/server/httpapi-query-schema-drift.test.ts
  • ./packages/sdk/js/script/build.ts generated only unrelated declaration-order churn; not committed.

Stack

@kitlangton kitlangton mentioned this pull request May 10, 2026
6 tasks
@kitlangton kitlangton force-pushed the cleanup/localize-id-prefixes branch from cf67a3f to 075cbb9 Compare May 10, 2026 03:33
@kitlangton kitlangton requested a review from adamdotdevin as a code owner May 10, 2026 03:33
@kitlangton kitlangton force-pushed the cleanup/openapi-diff-message-query branch from f7513af to 8308fbc Compare May 10, 2026 03:33
@kitlangton kitlangton force-pushed the cleanup/openapi-diff-message-query branch from 8308fbc to 729bf3e Compare May 10, 2026 15:43
@kitlangton kitlangton changed the base branch from cleanup/localize-id-prefixes to dev May 10, 2026 15:43
@kitlangton kitlangton mentioned this pull request May 10, 2026
3 tasks
@kitlangton kitlangton merged commit 49ee3ba into dev May 10, 2026
16 of 20 checks passed
@kitlangton kitlangton deleted the cleanup/openapi-diff-message-query branch May 10, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant