Skip to content

Conversation

@chrisghill
Copy link
Member

@chrisghill chrisghill commented Oct 23, 2025

This PR refactors the GraphQL API to move bundle information from being nested under manifest to being a direct field on the package query response. The change affects how bundle data is accessed throughout the codebase.

Key changes:

  • Moved Bundle field from Manifest type to Package type in GraphQL schema and API
  • Updated all references from pkg.Manifest.Bundle to pkg.Bundle in export and validation logic
  • Improved error handling in bundle creation with more descriptive error messages

This comment was marked as resolved.

Copy link
Member

@coryodaniel coryodaniel left a comment

Choose a reason for hiding this comment

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

Looks good, linter is a red herring.

@chrisghill chrisghill merged commit 15e80f0 into main Oct 23, 2025
3 of 4 checks passed
@chrisghill chrisghill deleted the chris/hotfix/remove-bundle-from-manifest-query branch October 23, 2025 19:40
@chrisghill
Copy link
Member Author

Looks good, linter is a red herring.

Yeah, linter "updated" and deprecated the old version. After updating its a ton of noise...

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.

3 participants