chore(main): release sql_studio 0.4.0 - #1
Closed
github-actions[bot] wants to merge 33 commits into
Closed
github-actions[bot] wants to merge 33 commits into
github-actions[bot] wants to merge 33 commits into
Conversation
- Adds `docs/` with `architecture.md`, `security.md`, `code_of_conduct.md`, and `contributing.md`, each in English, Spanish, and Brazilian Portuguese. - Replaces the root `CONTRIBUTING.md`, which only covered the commit convention, with `docs/contributing.md`, documenting setup, the actual scope vocabulary, the four CI jobs, and the release-please flow. - Adds a Documentation section to all three READMEs linking the new documentation. - Completes the Scripts table with `verify.sh`, `check_l10n.sh`, and `check_coverage.sh`.
- Uses `database.name` instead of `database.label` when setting the active database from a drawer card tap. - Keeps `activeDatabase` consistent with the deletion cleanup check and `SqlExecutionService`, both of which key database files by name.
- Adds `renovate.json` for weekly dependency update PRs with a dependency dashboard and `build(deps):` commits. - Pins the `act` runner image in `.actrc` for local workflow testing. - Adds `.github/pull_request_template.md` with a checklist matching the repository's conventions, including l10n, `Result<T>`, `AppLogger`, and `verify.sh`.
- Adds a "Testing workflows locally" section covering `.actrc` and how `act pull_request` mirrors CI. - Adds a "Dependency updates" section explaining Renovate's schedule and commit prefix from `renovate.json`. - Mirrors both sections in the `es` and `pt-BR` translations.
- Adds `codecov.yml`, ignoring `lib/l10n/**` so pull request coverage comments match the exclusions in `scripts/check_coverage.sh`. - Sets the project coverage target to the same 91% floor enforced by the script.
- Adds `.githooks/pre-push`, running `./scripts/verify.sh` before pushes that carry commits and exiting cleanly for branch deletions. - Documents the hook in `docs/contributing.md` alongside the existing `git config core.hooksPath` setup line in all three languages.
- Adds a second badge row with CI status, Codecov, very_good_analysis, and MIT license badges to `README.md`, `README.es.md`, and `README.pt-BR.md`.
- Wires `theme_persists_test.dart`, `workspace_layout_persists_test.dart`, and `favorite_database_persists_test.dart` into the integration job. - These suites were added in `8e84459` but never added to the emulator script, so CI ran 6 of the 9 flow suites and left these three without CI coverage.
- Adds an Architecture section covering feature scopes, state management, and cross-cutting code, with a link to `docs/architecture.md`. - Adds a Testing section covering the test file count, `integration_test` suites, enforced coverage floor, and local `genhtml` recipe. - Mirrors both sections in `README.es.md` and `README.pt-BR.md`.
- Adds a Tests table mapping each type of change to the corresponding test coverage and states the enforced 91% coverage floor. - Adds a Ripple effects checklist covering follow-up updates to `README.md`, `docs/architecture.md`, `docs/contributing.md`, and `pubspec.yaml` across all three languages. - Updates the stale coverage floor from 86% to 91% in the Commands section.
- Scaffolds the local `packages/app_ui` package following `music_app`'s `path:` dependency pattern, with its own `pubspec.yaml`, `analysis_options.yaml`, and `app_ui.dart` barrel export. - Moves `AppColors`, `AppDurations`, `AppRadii`, `AppShadows`, `AppSpacing`, `AppTheme`, and the `context.colors` extension into the new package. - Replaces deep core imports across all consumers with a single `package:app_ui/app_ui.dart` import while keeping class names unchanged.
- Adds an `app_ui` job to `ci.yaml` running format and analyze against the new package, without a test step because it has no tests until the next migration step. - Scans `packages/app_ui/pubspec.lock` with `osv-scanner` alongside the root lockfile.
- Moves `CardWidget`, `EmptyStateWidget`, `LoadingStateWidget`, `InputWidget`, `SwitchWidget`, and `PopupMenuSectionHeaderWidget`, along with their tests, into the new package. - Keeps `error_state_widget.dart` and `popup_menu_button_widget.dart` in the app for now: the former imports `ButtonWidget` and `DialogWidget`, which will move in a later step, while the latter reads `AppLocalizations` directly for its tooltip. - Ignores `packages/*/build/` and `packages/*/coverage/`, which were not covered by the root `.gitignore`'s anchored patterns after the package began producing its own build and test output.
- Adds test and coverage steps to the `app_ui` job with an interim 70% floor, measured at 72.6%, while untested token classes account for most of the gap. - Rewrites `app_ui` lcov paths as repository-relative paths before upload, matching the root job's existing path format. - Splits `codecov.yml` into `app` and `app_ui` flags with 91% and 70% targets, respectively, preserving each flag's coverage between runs.
- Moves `ButtonWidget`, `LoadingButtonWidget`, `resolveButtonStyle`, and `clampTextScaling`, along with their tests, into the new package. - Keeps `cancel_button_widget.dart` in the app because it reads `AppLocalizations.of(context).cancel` for its label and calls `Navigator.pop`, coupling it to app-specific behavior. - Keeps `error_state_widget.dart` in the app because it depends on `DialogWidget`, which will move with the dialogs in the next step.
- Raises the enforced `app_ui` coverage floor from 70% to 79%. - Buttons landed with tests, bringing real coverage from 72.6% to 81.7%. - Keeps a small margin below the measured coverage instead of locking in the exact 81.7% value.
- Moves `DialogWidget`, `ConfirmationDialogWidget`, `InputDialogWidget`, `ErrorDialogWidget`, `ErrorStateWidget`, and the `list_extension` helper, along with their tests, into the new package. - Adapts `ConfirmationDialogWidget` and `InputDialogWidget` to accept a `cancelButton` widget instead of hardcoding `CancelButtonWidget`, which stays in the app due to its `AppLocalizations` and `Navigator` coupling. - Adapts `ErrorDialogWidget` to accept `title` and `dismissLabel` instead of reading `AppLocalizations` directly. - Removes the localized "required" default from `InputDialogWidget`'s optional validator. - Updates every call site with the new required parameters.
- Adds `url_launcher_platform_interface` as a dev dependency and a `FakeUrlLauncherPlatform` test helper, allowing the `ErrorDialogWidget` shown when opening a link fails to be tested without a real platform channel. - Covers the failure branch in `about_screen_test.dart` and `settings_screen_test.dart`, which were previously untested. - Restores root coverage above the 91% floor after moving the dialogs out of the app caused coverage to fall below the threshold.
- Adds a "Design system (packages/app_ui)" section to `docs/architecture.md` in all three languages, explaining the package layout and coupling boundary rule: no `AppLocalizations`, providers, or routing. - Fixes a stale reference to design tokens living under `lib/src/core/`. - Updates `docs/contributing.md` in all three languages with the current five-job CI table, design-system tip, two-flag Codecov coverage report, and note that `packages/app_ui` needs its own local gate run. - Updates the README Architecture and Testing sections in all three languages with current test counts and the two-flag Codecov upload. - Adds a "Where code goes" note to `CLAUDE.md`, directing presentation-agnostic widgets to `packages/app_ui`, and documents an `app-ui` commit scope.
- Adds a `verify_package` helper to `scripts/verify.sh`, running format, analyze, test, and the coverage gate inside a given package directory. - Calls it for `packages/app_ui` with its own 79% floor, immediately after the root app's checks. - Prevents package-scoped changes from passing the local gate without exercising `packages/app_ui`, while CI's dedicated `app_ui` job could still fail.
- Adds `packages/` to both the tracked and untracked `git ls-files` scopes in `workspace_hash.sh`. - Ensures edits to `packages/app_ui` change the workspace hash, allowing the Stop hook's verify-stamp comparison to detect unverified app_ui changes instead of letting a turn end without verifying them.
- Adds `permissions.allow` for the project's own scripts and the `fvm flutter`/`dart` command family required by the project conventions, so routine verify, test, and format runs do not prompt for approval each time. - Adds `$schema` for editor completion. - Adds a timeout and `statusMessage` to both existing hooks.
- Adds `.claude/settings.local.json`, a personal permissions override, and `.claude/worktrees/`, created by isolated agent runs, to `.gitignore`. - Keeps the existing verify-stamp entry alongside the new Claude Code state exclusions.
- Changes `dart.flutterSdkPath` from the pinned `.fvm/versions/3.41.6` path to the `.fvm/flutter_sdk` symlink maintained by FVM. - Prevents the next `.fvmrc` version bump from silently breaking editor integration for every contributor.
- Adds `build/**` to the analyzer `exclude` list in the root and `packages/app_ui` `analysis_options.yaml`. - Prevents stale generated files under `build/` from appearing in `flutter analyze` output when they are unrelated to the project's source.
- Reorders the language links to English, Español, Português (BR) in `README.md`, `README.es.md`, and `README.pt-BR.md`.
github-actions
Bot
force-pushed
the
release-please--branches--main--components--sql_studio
branch
from
September 12, 2026 20:07
60a20b6 to
5625f71
Compare
- Passes `packages/app_ui/coverage/lcov.info` to `check_coverage.sh` in the `app_ui` job's coverage step.
github-actions
Bot
force-pushed
the
release-please--branches--main--components--sql_studio
branch
from
September 12, 2026 21:28
5625f71 to
9850047
Compare
- Adds `paint()` coverage for `TableRelationPainter` covering a missing source/target rect, a relation with no selection, a relation touching the selected table (highlighted), and one that does not (dimmed).
github-actions
Bot
force-pushed
the
release-please--branches--main--components--sql_studio
branch
from
September 13, 2026 21:48
9850047 to
77368d3
Compare
- Reorders sections to the fixed order: About, Preview, Features, Tech Stack, Architecture, Getting Started, Scripts, Testing, Documentation, Contributing, Security, License, and Author. - Renames Screenshots to Preview and Built With to Tech Stack. - Adds a Security section linking to the localized `docs/security*.md` files. - Applies the same structure to `README.md`, `README.pt-BR.md`, and `README.es.md`.
github-actions
Bot
force-pushed
the
release-please--branches--main--components--sql_studio
branch
from
September 20, 2026 10:28
77368d3 to
36b48ca
Compare
- Documents that `verify.sh` also gates `packages/app_ui` against its own coverage floor. - Adds `LocalStateService` to the architecture layout. - Documents that external links open in the system browser. - Replaces em dashes with plain punctuation across the three languages.
- Replaces four doc comments and one test description with commas, colons, or semicolons. - Follows the convention that comments and commit messages contain no em dashes. - Makes no behavior changes.
- Adds two-space indentation to nested HTML in the badge rows, language switcher, header summary, and Preview gallery across all three READMEs. - Updates the Scripts table's `verify` row to state that the gate covers `packages/app_ui` as well as the app.
github-actions
Bot
force-pushed
the
release-please--branches--main--components--sql_studio
branch
from
September 20, 2026 13:56
36b48ca to
0033ddc
Compare
dariomatias-dev
force-pushed
the
main
branch
from
September 27, 2026 19:52
143c491 to
7fefccd
Compare
dariomatias-dev
deleted the
release-please--branches--main--components--sql_studio
branch
September 27, 2026 19:53
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This release is too large to preview in the pull request body. View the full release notes here: https://github.com/dariomatias-dev/sql_studio_app/blob/release-please--branches--main--components--sql_studio--release-notes/release-notes.md