Move agents/skills -> .agents/skills (post-release; resolve the bench-branch divergence with it)

未关闭
#1,925 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
4/5
预计耗时
3-5 天
新手友好度
35/100
Issue 类型
重构
描述清晰度
描述清楚
活跃度
停滞
技术栈
python
领域
tooling

调研方向

Read AGENTS.md, CLAUDE.md, the self-references in review/SKILL.md and plan/SKILL.md, and compare the skill copies on master and bench/gfql-pokec-polars-lanes. The issue says the bench branch’s +52-line review-protocol edit has already landed via #1924; reconcile the copies, update the documented paths, and verify a fresh clone exposes skills to Codex and Claude Code without manual steps.

由索引模型根据 Issue 内容生成。

描述

Owner decision: .agents/skills/ is the universal convention and pygraphistry should adopt it. Post-release — PRs stay on the current agents/ layout until the GFQL stack lands.

Current state (measured)

  • master tracks agents/skills/ — 3 SKILL.md files.
  • bench/gfql-pokec-polars-lanes tracks .agents/skills/ and has no agents/. The rename happened in 02fc3d1b1, an unrelated benchmark commit — an accidental divergence, not a deliberate migration. It will conflict or double-apply if that branch merges, so fold it into this work rather than letting a bench commit define the repo layout.
  • CLAUDE.md currently documents the opposite of the target ("Canonical repo skills are in agents/skills/", plus a ln -s agents .agents step).

Blast radius: small — 8 references in 4 files

AGENTS.md, CLAUDE.md, and self-references inside the skills (review/SKILL.md ×3, plan/SKILL.md ×1), e.g. agents/skills/review/SKILL.md#vectorization--engine-compatibility. Those in-document anchors are the easy miss because they still read correctly after a rename. No CI or bin/ script references the path, so this cannot break a lane — only documentation links.

Plan (single PR)

  1. git mv agents .agents as one rename-only commit so --find-renames keeps history legible.
  2. Fix the 8 references; then git grep agents/skills must return zero hits that are not .agents/skills.
  3. Rewrite CLAUDE.md's "Skill path conventions" block: .agents/skills/ canonical, keep the .claude/skills symlink step for Claude Code, drop the now-circular ln -s agents .agents.
  4. Reconcile the bench branch: diff the two copies and fold in anything it added (it currently carries a +52-line review-protocol edit that was applied there by mistake and has since been landed on master properly via #1924).
  5. Verify a fresh clone gives both Codex and Claude Code a working skills path with no undocumented manual step.

Why not now

A tree-wide rename conflicts with every open branch touching those files. ~15 stacked PRs are open and #1924 is pending against agents/skills/review/SKILL.md; renaming first would force a rebase for zero benefit.

主要语言
Python
星标
2.6k
派生
229
平均合并
1 天 5 小时
30 天内合并 PR
99

环境准备

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

graphistry/pygraphistry 的其他 Issue

查看 graphistry/pygraphistry 的全部 Issue

相似的 Issue

更多 Python Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。