chore: tighten AI workflow skills and align project docs#25
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR standardizes the repo’s AI workflow documentation and templates, ensuring skill frontmatter is uniform and the architecture guidance consistently routes data flow through Riverpod use cases/providers (with repositories only when already present or explicitly justified).
Changes:
- Normalize skill metadata by backfilling
allowed-tools:andmodel:in skill frontmatter and documenting these as required inAGENTS.md. - Update
techspec/taskstemplates and skill guidance to de-emphasize a default repository layer in favor of use cases/providers. - Align docs and references: README ToC labels,
.claude/CLAUDE.mdcommand list,project-setupchecklist item references, andcreate-prscreenshot placeholder.
Reviewed changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Align ToC entry labels with section headers. |
| ai/templates/techspec.md | Update architecture/template guidance and examples to use use cases/providers by default. |
| ai/templates/task.md | Adjust task template prompts to reference use cases/providers vs repositories. |
| ai/templates/task-list.md | Update Phase 1 wording to “use cases/providers” instead of repository-first phrasing. |
| ai/skills/upgrade/SKILL.md | Add/standardize allowed-tools + model frontmatter. |
| ai/skills/techspec/SKILL.md | Update guidance to prefer use cases/providers and limit repository abstraction to justified cases. |
| ai/skills/tasks/SKILL.md | Update data-layer phase guidance to Riverpod use cases/providers. |
| ai/skills/secrets-bootstrap/SKILL.md | Add/standardize allowed-tools + model frontmatter. |
| ai/skills/release-prepare/SKILL.md | Add/standardize allowed-tools + model frontmatter. |
| ai/skills/release-builds/SKILL.md | Add/standardize allowed-tools + model frontmatter. |
| ai/skills/project-setup/SKILL.md | Add/standardize frontmatter; fix README checklist item references for platform cleanup/Firebase. |
| ai/skills/pr-review/SKILL.md | Add/standardize allowed-tools + model frontmatter. |
| ai/skills/feature-screen/SKILL.md | Add/standardize allowed-tools + model frontmatter. |
| ai/skills/feature-data-flow/SKILL.md | Add/standardize allowed-tools + model frontmatter. |
| ai/skills/create-pr/SKILL.md | Normalize screenshot placeholder text in PR body template. |
| AGENTS.md | Require allowed-tools and model (and document user-invocable) for new skills. |
| .claude/CLAUDE.md | Refresh slash-command listing to reflect current commands/skills. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Chores
allowed-tools:andmodel:on all 18 skills so SKILL.md frontmatter is uniform, and updateAGENTS.md"Creating a new skill" to require these fields for new skills..claude/CLAUDE.mdslash-command listing and align README ToC labels with their section headers.project-setupskill to reference the right README item number for the Firebase step, and normalize thecreate-prscreenshot placeholder.Refactor
tasks/techspecskills and templates; data flow now goes through use cases and providers, with repositories only when already present or explicitly justified.