Skip to content

chore(deps): clean up package dependency scopes#112

Merged
leoafarias merged 1 commit into
mainfrom
chore/dependency-hygiene
May 28, 2026
Merged

chore(deps): clean up package dependency scopes#112
leoafarias merged 1 commit into
mainfrom
chore/dependency-hygiene

Conversation

@leoafarias
Copy link
Copy Markdown
Collaborator

Summary

  • Move path in ack_generator to dev_dependencies because it is only used by tests/tooling.
  • Move firebase_ai in ack_firebase_ai to dev_dependencies because package library code does not import it directly.
  • Add ack_annotations to the example dependencies because example/lib imports it directly.

Validation

  • dart analyze . --fatal-warnings --fatal-infos for all workspace packages
  • dependency_validator --verbose -C <package> for each package
  • dart test in packages/ack, packages/ack_generator, packages/ack_json_schema_builder, and example
  • flutter test in packages/ack_firebase_ai
  • cd tools && npm install --silent && npx depcheck --json

Notes

  • Melos orchestration is blocked in this workspace because pubspec.yaml points to .fvm/flutter_sdk, which is not present locally, so validation used direct package-level commands.

@docs-page
Copy link
Copy Markdown

docs-page Bot commented May 28, 2026

To view this pull requests documentation preview, visit the following URL:

docs.page/btwld/ack~112

Documentation is deployed and generated using docs.page.

@leoafarias leoafarias merged commit e605afa into main May 28, 2026
1 check passed
@leoafarias leoafarias deleted the chore/dependency-hygiene branch May 28, 2026 18:34
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