mirror of
https://bitbucket.org/siakitem/my-pi.git
synced 2026-10-26 10:45:22 +00:00
云服务部-叶林立
312bb2ba5a
从官方 Git 仓库导入 chandra447/pi-hermes-memory 在 tag v0.9.10 （df624056a2e8c68bc89ba0a81e1c120856ec9665，2026-10-02）的源码快照。 本次仅导入源码，尚未接入组合包：package.json、加载顺序、依赖与权限策略 均未改动。 - 原样导入 src/、tests/、docs/、scripts/、README、CHANGELOG、PLAN、 AGENTS 与 LICENSE；该快照共 156 个上游文件。 - 剔除嵌套 .git、上游 .github 工作流、.ralph/、.claude/、node_modules、 覆盖率与构建产物。 - 新增中文 UPSTREAM.md，记录来源，以及启用前必须完成的待决事项： 在 pi-compaction 之前注册加载顺序、better-sqlite3 版本与根 allowScripts 对齐、strip-ansi 解析确认。 - 说明选择 tag 而非 main 的原因：导入时上游 main 已领先至 24fcd8ad26fc27badb8d7dbaf8034645de07b893（2026-10-11），选 tag 是为 对应一个已发布、可复现的版本。 尚未接线。启用前还需反转 tests/extension-load.test.ts 中"不得加载 Hermes"的断言，并同步 pi-tool-search 分组、权限策略与文档。 验证：快照 156 个上游文件全部导入，目录内仅多出 UPSTREAM.md 一份； 已确认与上游 main 内容存在差异，证明取的是 tag 而非 main。
61 lines
2.6 KiB
TypeScript
61 lines
2.6 KiB
TypeScript
/**
|
|
* Contract test for Pi's skill index (follow-up to #244 / #249).
|
|
*
|
|
* Pi discovers skills by the frontmatter `description` alone — the SKILL.md
|
|
* body never reaches the rendered `<available_skills>` block. This test pins
|
|
* that contract with Pi's own loader through the public API, so a change in
|
|
* how Pi indexes skills fails here instead of silently degrading discovery.
|
|
*/
|
|
|
|
import { describe, it } from "node:test";
|
|
import * as assert from "node:assert/strict";
|
|
import * as fs from "node:fs/promises";
|
|
import * as os from "node:os";
|
|
import * as path from "node:path";
|
|
import { formatSkillsForPrompt, loadSkills } from "@earendil-works/pi-coding-agent";
|
|
|
|
async function writeSkill(dir: string, slug: string, description: string, whenToUse: string): Promise<void> {
|
|
const skillDir = path.join(dir, slug);
|
|
await fs.mkdir(skillDir, { recursive: true });
|
|
await fs.writeFile(
|
|
path.join(skillDir, "SKILL.md"),
|
|
`---\nname: ${slug}\ndescription: "${description}"\n---\n## When to Use\n${whenToUse}\n`,
|
|
"utf-8",
|
|
);
|
|
}
|
|
|
|
describe("Pi skill index contract", () => {
|
|
it("surfaces description triggers in the index and never body-only triggers", async () => {
|
|
const root = await fs.mkdtemp(path.join(os.tmpdir(), "pi-skill-discovery-contract-"));
|
|
try {
|
|
const skillsDir = path.join(root, "skills");
|
|
await writeSkill(
|
|
skillsDir,
|
|
"branch-check-description",
|
|
"Check git branch state. Use when asked to check branches or which branches are unmerged.",
|
|
"Use when the user asks to compare local and remote branch state.",
|
|
);
|
|
await writeSkill(
|
|
skillsDir,
|
|
"branch-check-body-only",
|
|
"Check git branch state: local vs remote counts, unmerged branches.",
|
|
"Triggered when asked to check branches. Keywords: branch check, unmerged.",
|
|
);
|
|
|
|
const result = loadSkills({ cwd: root, agentDir: root, skillPaths: [skillsDir], includeDefaults: false });
|
|
const index = formatSkillsForPrompt(result.skills);
|
|
|
|
assert.match(index, /<name>branch-check-description<\/name>/, "both skills are indexed by name");
|
|
assert.match(index, /<name>branch-check-body-only<\/name>/, "both skills are indexed by name");
|
|
assert.match(index, /Use when asked to check branches/, "description-carried triggers are discoverable");
|
|
assert.doesNotMatch(index, /Keywords: branch check, unmerged/, "body-only triggers never reach the index");
|
|
} finally {
|
|
try {
|
|
await fs.rm(root, { recursive: true, force: true });
|
|
} catch {
|
|
// Windows may keep the directory busy briefly; the temp dir is disposable.
|
|
}
|
|
}
|
|
});
|
|
});
|