Skip to content

chore: tighten AI workflow skills and align project docs#25

Merged
michalurbanek merged 1 commit into
masterfrom
mu/chore/ai_flows_polish
May 19, 2026
Merged

chore: tighten AI workflow skills and align project docs#25
michalurbanek merged 1 commit into
masterfrom
mu/chore/ai_flows_polish

Conversation

@michalurbanek
Copy link
Copy Markdown
Collaborator

Chores

  • Backfill allowed-tools: and model: on all 18 skills so SKILL.md frontmatter is uniform, and update AGENTS.md "Creating a new skill" to require these fields for new skills.
  • Refresh .claude/CLAUDE.md slash-command listing and align README ToC labels with their section headers.
  • Fix project-setup skill to reference the right README item number for the Firebase step, and normalize the create-pr screenshot placeholder.

Refactor

  • Drop the default repository layer from tasks / techspec skills and templates; data flow now goes through use cases and providers, with repositories only when already present or explicitly justified.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

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: and model: in skill frontmatter and documenting these as required in AGENTS.md.
  • Update techspec/tasks templates 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.md command list, project-setup checklist item references, and create-pr screenshot 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.

@michalurbanek michalurbanek merged commit 5f7997f into master May 19, 2026
5 checks passed
@michalurbanek michalurbanek deleted the mu/chore/ai_flows_polish branch May 19, 2026 12:02
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.

2 participants