feat(pipeline): give docs and skill-file changes an area in feature-factory
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 5/5
- 预计耗时
- 一周以上
- 新手友好度
- 35/100
- Issue 类型
- 功能
- 描述清晰度
- 基本清楚
- 活跃度
- 活跃
调研方向
Start by reading .claude/skills/feature-factory/SKILL.md and the linked decisions in #400, #437, #448, and #451. Grooming must first record which of the three proposed approaches the owner chooses; then update the skill and, for option 1, add .claude/agents/docs-builder.md. Done means the scope boundaries and pipeline behavior match the chosen approach, a docs-only run reaches stop 3, and a later real run is recorded on the issue.
由索引模型根据 Issue 内容生成。
描述
Follow-up to #438 (the merged feature-factory skill). Found on the pipeline runs for #456 and #444. Not groomed: it needs one decision (below) before a builder starts. Branch from main and open the PR against main.
Problem
The pipeline knows three areas: api, web and assistant. The Scope block lists which of them change (.claude/skills/feature-factory/SKILL.md, stage 3 step 1) and stage 4 runs one builder per listed area. A change that touches only docs or Claude config files (docs/, .claude/skills, .claude/agents, .claude/rules, README files) has no area and no builder.
Both runs hit it, and both handled it by hand:
- #456 (a skill edit): "The SKILL.md edit was applied by a general-purpose agent, since no builder covers a
.claudeskill file." https://github.com/nevridge/TaskFlow/issues/456#issuecomment-6716756859 - #444 (a docs edit): "No feature-factory builder owns
docs/; the owner chose a one-off general-purpose agent for the docs edit." https://github.com/nevridge/TaskFlow/issues/444#issuecomment-6716895666
Costs: an extra decision for the owner at stop 2 every time, a different answer possible each run, and no path boundary. The three builders each say "never touches other areas"; a general-purpose agent has no such limit, so nothing stops it editing settings.json, hooks, compose files or CI.
Constraint from #400: claude-integration-advisor deliberately does not cover repo .claude/, CLAUDE.md or the project agents, because "tooling edits [are] made by hand, not feature work" (decision IA3). This issue has to either respect that or revisit it explicitly.
Decide in grooming
- A
docsarea built by a newdocs-builderagent with a path allowlist, like the other builders. Most consistent, but an eighth agent after #437 consolidated to seven. - A
docsarea defined in the skill and built by a general-purpose agent, with the path allowlist and "never touches other areas" stated in the skill's prompt, as #400 does for other project needs. No new agent file. This is the recommendation. - No area: the pipeline refuses a docs-only diff, names the reason at preflight or after the Scope block, and tells the owner to use a plain builder session from the issue (as #448 and #451 were). Consistent with IA3, but the owner loses the stops, the validator and the notes for these changes, which they have been using.
Change (for options 1 or 2)
Edit .claude/skills/feature-factory/SKILL.md (and add .claude/agents/docs-builder.md for option 1):
- The Scope block can list a fourth area,
docs:docs/**,.claude/skills/**,.claude/agents/**,.claude/rules/**, and*.mdfiles outsidedocs/(README files,CLAUDE.md). - Stage 4 builds it like the others (one commit,
build-docs.md), in dependency order after the code areas. - Boundaries: never
.claude/settings*.json, hooks,.github/**, compose files,.envor any source or test file. An agent file edit keeps its frontmatter valid YAML (check withjs-yaml, as #448 did). - Stages 5 and 6 need no change: the verifier runs no gates when no code area changed, and the validator checks the final diff against the brief. State that a docs-only run skips the code gates, so the verifier does not report "environment failure".
- The owner is not asked at stop 2 which agent to use.
Acceptance criteria
- The owner decision above is recorded on this issue.
- A docs-only or skill-only issue run through
/feature-factoryreaches stop 3 without a stop 2 question about which agent builds it. - The builder for that area cannot touch the paths excluded above, and says so in its prompt.
- The change does not alter the stops, the budgets or the three existing areas.
- Verified on the next real docs-only or skill-only pipeline run, recorded on this issue.
Tests
No tests apply (skill and agent prompts). Say so in the PR body.
Out of scope
- The
claude-integration-advisorscope (IA3 stands unless the owner says otherwise here). - Changes to
pr-triageor the user-level agents.
- 主要语言
- C#
- 星标
- 1
- 派生
- 0
- 平均合并
- 2 小时 2 分钟
- 30 天内合并 PR
- 98
环境准备
- 提供 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
nevridge/TaskFlow 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 65/100
维护者通常 1 天内回复
-
tech-debt
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
查看 nevridge/TaskFlow 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 74/100
juandresrodca/AdminForge#54 ·
-
要トリアージ 辞書
难度 1/5 1-3 小时 新手友好度 86/100
yksr-melt/Meltype#1625 · 1 个 reaction ·
维护者通常 1 天内回复
-
Docs typo未关闭
难度 1/5 1 小时以内 新手友好度 78/100
MonoGame/docs.monogame.github.io#328 · 1 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 70/100
space-wizards/RobustToolbox#7197 ·
维护者通常 1 天内回复
-
documentation high priority
难度 2/5 1-3 小时 新手友好度 76/100
lachydotmcg/capri-desktop#201 ·
维护者通常 1 天内回复