Repository navigation
chore: add unicoder.yaml with VGV preset - #168
Open
erickzanardo wants to merge 3 commits into
Open
erickzanardo wants to merge 3 commits into
erickzanardo wants to merge 3 commits into
Conversation
Configures the Unicoder bot from the repo with the VGV Best Practices preset: Flutter pinned to the same 3.47 line CI uses, Very Good CLI, and the VGV Claude plugins. Adds "unicoder" to the spell-check words so the file and PR title pass cspell. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Contributor
Contributor
There was a problem hiding this comment.
This is a small, well-documented configuration PR that adds a unicoder.yaml bot config using the VGV Best Practices preset, plus a cspell dictionary entry. The YAML is clean and the key choices are sound:
Positives
version: "3.47"is correctly quoted, avoiding YAML interpreting it as a float, and aligns with theflutter_version: "3.47.x"pin in.github/workflows/example.yaml(both resolve to the newest 3.47.x patch).- The
very_good_clitool and the two Claude plugins/marketplace are declared consistently. - The PR description clearly explains the per-section override semantics and the CI-version coupling.
Minor observation
- The stated rationale for adding
unicoderto the cspell words does not appear to hold: the spell-check workflow only scans**/*.md(includes: **/*.md), sounicoder.yamlis never spell-checked, and the wordunicoderdoes not appear in any markdown file in the repo. The cspell entry is therefore harmless but likely unnecessary unless a future.mdfile (or a changed spell-check scope) references it.
No correctness, security, or lifecycle issues found. Approvable as-is; the cspell note is optional cleanup.
erickzanardo
commented
Oct 16, 2026
erickzanardo
commented
Oct 16, 2026
erickzanardo
enabled auto-merge (squash)
October 16, 2026 17:54
The spell-check workflow also checks the PR title, which contains "unicoder"; dropping the word fails that step. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
ricardodalarme
approved these changes
Oct 16, 2026
ricardodalarme
approved these changes
Oct 17, 2026
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
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.
Adds a
unicoder.yamlso the Unicoder bot configures this repo from the branch itself, using the VGV Best Practices preset:"3.47", which resolves to the newest 3.47.x patch. That's the same line CI pins (flutter_version: "3.47.x"in.github/workflows/example.yaml). The bot doesn't accept the.xsuffix, so the major.minor form is used instead.1.5.0(latest).vgv-ai-flutter-pluginandvgv-wingspanfromVeryGoodOpenSource/very-good-claude-code-marketplace.Each section the file declares replaces the dashboard's value for that section, and sections it leaves out (such as hooks) keep their dashboard values. Very Good Blueprints is part of the preset but can't be set from the file, so it stays in the dashboard.
Also adds
unicoderto the cspell words, so the new file and this PR title pass the spell check.When CI moves to a new Flutter version, update the
versionhere to match.馃 Generated with Claude Code