Skip to content

chore: sync openapi.json with upstream API#304

Merged
sachiniyer merged 1 commit into
mainfrom
devin/1779842731-sync-openapi-schema
May 27, 2026
Merged

chore: sync openapi.json with upstream API#304
sachiniyer merged 1 commit into
mainfrom
devin/1779842731-sync-openapi-schema

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot commented May 27, 2026

Summary

Syncs openapi.json with the upstream API at https://api.detail.dev/public/v1/openapi.json using cargo xtask generate-openapi.

Changes from upstream:

  • New asanaStateName field added to BugReview properties
  • "asana" added to the integration source enum

The API client is generated at compile time via progenitor::generate_api!, so no other file changes are needed.

Review & Testing Checklist for Human

  • Verify the new asanaStateName field and asana enum value are expected upstream additions

Notes

Generated via cargo xtask generate-openapi. All checks pass: cargo xtask check, cargo fmt --check, cargo clippy -- -D warnings, cargo test (295 tests).

Link to Devin session: https://app.devin.ai/sessions/cea71513bc1840c69f62012799a35cb1
Requested by: @sachiniyer


Open in Devin Review

Summary by cubic

Synced openapi.json with the upstream API at https://api.detail.dev/public/v1/openapi.json. Adds BugReview.asanaStateName and asana to the integration source enum; the client updates at build via progenitor::generate_api!.

Written for commit 2d08e8a. Summary will update on new commits. Review in cubic

Co-Authored-By: Sachin Iyer <siyer@detail.dev>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

Original prompt from Sachin

Can you sync up the openapi schema with teh remote. Basically just update with xtasks

@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Copy Markdown
Contributor Author

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

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review

Comment thread openapi.json
"review",
"linear",
"jira",
"asana",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

📝 Info: BugSource not re-exported — new variant has no downstream impact

The BugSource type is notably absent from the re-export list at src/api/types.rs:7-12. It exists only as a field type within BugReview.source (which is itself optional). No code in the repository pattern-matches on BugSource variants, so adding "asana" is purely additive with zero risk of breaking existing match arms. If a future PR needs to display the source (e.g., in a card view), it would need to handle the new variant at that point.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

@sachiniyer sachiniyer merged commit f71b563 into main May 27, 2026
13 checks passed
@sachiniyer sachiniyer deleted the devin/1779842731-sync-openapi-schema branch May 27, 2026 00:53
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