Skip to content

fix: vendor updated openapi.json so BugSource accepts new backend variants - #255

Merged
sachiniyer merged 1 commit into
mainfrom
siyer/vendor-openapi-bugsource-update
Apr 21, 2026
Merged

sachiniyer merged 1 commit into
mainfrom
siyer/vendor-openapi-bugsource-update

Conversation

@sachiniyer

@sachiniyer sachiniyer commented Apr 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • The backend now emits review.source values of jira, github_issue, and bug_fix_check on real bug responses, but the vendored openapi.json only declared review and linear.
  • Progenitor's generated BugSource enum therefore rejected the actual payload, causing every command that deserializes a review to fail with Invalid Response Payload: unknown variant 'github_issue':
    • detail bugs list --status resolved
    • detail bugs list --status dismissed
    • detail bugs list --vulns / --introduced-by (they pull all statuses internally)
    • detail bugs show <bug_id> against any review-bearing bug
  • Ran cargo xtask generate-openapi to pull the updated spec from https://api.detail.dev/public/v1/openapi.json. The only delta is the three new BugSource variants; progenitor will regenerate the matching Rust enum on build.

Test plan

  • cargo build succeeds (progenitor regenerates BugSource with the new variants)
  • cargo run -- bugs list usedetail/cli --status resolved --format json now returns valid JSON instead of Failed to fetch bugs from repository
  • cargo run -- bugs list usedetail/cli --status dismissed --format json same
  • CI integration tests (which exercise bugs_list_all_statuses against the live API) go green

Customer impact

The current 0.2.0 release is broken for any repo with reviews sourced from the new backend integrations. Recommend cutting a patch release after this lands.

🤖 Generated with Claude Code


Open in Devin Review

…iants

The backend now emits `review.source` values of `jira`, `github_issue`,
and `bug_fix_check`, but the vendored `openapi.json` only declared
`review` and `linear`. Progenitor's generated `BugSource` enum therefore
rejected real responses, breaking every command that deserializes a
review — including `bugs list --status resolved`, `bugs list --status
dismissed`, and `bugs show` against any review-bearing bug.

Run `cargo xtask generate-openapi` to pull the updated spec from
`https://api.detail.dev/public/v1/openapi.json`. The only delta is the
three new `BugSource` variants.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@sachiniyer
sachiniyer temporarily deployed to integration-tests April 21, 2026 10:08 — with GitHub Actions Inactive

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@sachiniyer
sachiniyer merged commit 55a0ff7 into main Apr 21, 2026
13 checks passed
@sachiniyer
sachiniyer deleted the siyer/vendor-openapi-bugsource-update branch April 21, 2026 10:11
@sachiniyer sachiniyer mentioned this pull request Apr 21, 2026
1 of 2 tasks
sachiniyer added a commit that referenced this pull request Apr 21, 2026
## Summary
Patch release shipping #255, which unbreaks `bugs list --status
resolved|dismissed` and `bugs show` for reviews sourced from the
backend's new `jira`, `github_issue`, and `bug_fix_check` integrations.

On merge, the release workflow will tag `v0.2.1` and publish platform
artifacts via cargo-dist.

## Test plan
- [x] `cargo build` succeeds with version 0.2.1
- [ ] Tag `v0.2.1` is created on merge and release workflow publishes
artifacts

🤖 Generated with [Claude Code](https://claude.com/claude-code)
<!-- devin-review-badge-begin -->

---

<a href="https://app.devin.ai/review/usedetail/cli/pull/256"
target="_blank">
  <picture>
<source media="(prefers-color-scheme: dark)"
srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1">
<img
src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1"
alt="Open in Devin Review">
  </picture>
</a>
<!-- devin-review-badge-end -->

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

This branch was previously deployed

1 inactive deployment
integration-tests — c0375e35 Deployed Apr 21, 2026 by sachiniyer via Integration #205
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