Skip to content

Conversation

@endersonmaia
Copy link
Contributor

This pull request removes GraphQL support from both the SDK and CLI packages, along with all related configuration, installation, and database setup. The changes simplify the codebase and deployment by eliminating unused dependencies and services.

Removal of GraphQL support:

  • Deleted the docker-compose-graphql.yaml file and all references to the GraphQL service from the CLI service definitions in rollups.ts. [1] [2]
  • Removed all GraphQL-related installation steps, migration code, and binaries from the packages/sdk/Dockerfile, including the creation of the graphql database and migration scripts. [1] [2] [3] [4]
  • Removed GraphQL version and migration variables from the Docker build configuration in docker-bake.hcl.

Package updates:

  • Updated the changeset files to note the removal of GraphQL support as a minor change for both @cartesi/sdk and @cartesi/cli. (.changeset/funny-humans-prove.md [1] .changeset/quick-fans-serve.md [2]

@endersonmaia endersonmaia self-assigned this Nov 10, 2025
@changeset-bot
Copy link

changeset-bot bot commented Nov 10, 2025

🦋 Changeset detected

Latest commit: ddc3f07

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@cartesi/sdk Minor
@cartesi/cli Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@endersonmaia endersonmaia requested a review from tuler November 10, 2025 14:41
@github-actions
Copy link
Contributor

github-actions bot commented Nov 10, 2025

Coverage Report for ./apps/cli

Status Category Percentage Covered / Total
🔵 Lines 23.26% 279 / 1199
🔵 Statements 23.16% 284 / 1226
🔵 Functions 24.63% 50 / 203
🔵 Branches 20% 112 / 560
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
apps/cli/src/exec/rollups.ts 0% 0% 0% 0% 43-543
Generated in workflow #680 for commit ddc3f07 by the Vitest Coverage Report Action

Copy link
Member

@tuler tuler left a comment

Choose a reason for hiding this comment

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

go-migrate left, check CI

@endersonmaia endersonmaia force-pushed the chore/remove-graphql-support branch from a59a0ff to ddc3f07 Compare November 10, 2025 17:21
@endersonmaia endersonmaia merged commit 929b8ea into prerelease/v2-alpha Nov 10, 2025
2 checks passed
@endersonmaia endersonmaia deleted the chore/remove-graphql-support branch November 10, 2025 17:24
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.

3 participants