Skip to content

feat: show Detail fix PR on bugs list and show views#319

Merged
tjdammann merged 1 commit into
mainfrom
tjdammann/bugs-fix-pr
Jul 24, 2026
Merged

feat: show Detail fix PR on bugs list and show views#319
tjdammann merged 1 commit into
mainfrom
tjdammann/bugs-fix-pr

Conversation

@tjdammann

@tjdammann tjdammann commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

What

Renders the Detail-generated fix PR (fixPr) on the bugs views, now that the public API exposes it.

  • bugs list (table + --format json): a compact Fix PR #<number> (<state>) row, alongside the existing Introduced / Linked Issues rows.
  • bugs show: a Fix PR #<number> (<state>) — <url> row, next to the per-issue rows.

fixPr is optional — bugs without a Detail fix PR simply omit the row (the field is only populated for repos with the bug-fix workflow enabled).

Changes

  • openapi.json: regenerated via cargo xtask generate-openapi — picks up the new optional Bug.fixPr + FixPr schema.
  • src/api/types.rs: re-export FixPr, add a format_fix_pr helper, add the Fix PR row to the Bug list card, plus unit tests.
  • src/commands/bugs.rs: add the Fix PR row (with URL) to render_bug_show.

Verification

  • cargo xtask check (openapi.json matches upstream, HELP.md up to date), cargo fmt --check, cargo clippy -- -D warnings, cargo build, cargo test — all green.
  • Smoke-tested the built binary against the live API: bugs list shows #13922 (open) / #13921 (closed), and bugs show shows #13599 (merged) — https://github.com/usedetail/detail/pull/13599 — across all three states.

Backend side (exposes the field): usedetail/detail#14560 (merged + deployed).

🤖 Generated with Claude Code


Open in Devin Review

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@aviator-app

aviator-app Bot commented Jul 23, 2026

Copy link
Copy Markdown

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged manually (without Aviator). Merging manually can negatively impact the performance of the queue. Consider using Aviator next time.


See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

@tjdammann
tjdammann temporarily deployed to integration-tests July 23, 2026 23:37 — 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 3 files

Re-trigger cubic

@tjdammann
tjdammann merged commit 79486ad into main Jul 24, 2026
14 of 15 checks passed
@tjdammann
tjdammann deleted the tjdammann/bugs-fix-pr branch July 24, 2026 19:21
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