Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
60a75f8159 | ||
|
|
b41cb8d3af | ||
|
|
d676635325 | ||
|
|
f02f9424a5 | ||
|
|
2302fbc843 | ||
|
|
c1c80f3cc8 |
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "ponytail",
|
"name": "ponytail",
|
||||||
"version": "4.4.0",
|
"version": "4.5.0",
|
||||||
"description": "Lazy senior dev mode. Forces the simplest, shortest solution that actually works: YAGNI, stdlib first, no unrequested abstractions.",
|
"description": "Lazy senior dev mode. Forces the simplest, shortest solution that actually works: YAGNI, stdlib first, no unrequested abstractions.",
|
||||||
"author": {
|
"author": {
|
||||||
"name": "Dietrich Gebert",
|
"name": "Dietrich Gebert",
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "ponytail",
|
"name": "ponytail",
|
||||||
"version": "4.4.0",
|
"version": "4.5.0",
|
||||||
"description": "Lazy senior dev mode. Forces the simplest, shortest solution that actually works: YAGNI, stdlib first, no unrequested abstractions.",
|
"description": "Lazy senior dev mode. Forces the simplest, shortest solution that actually works: YAGNI, stdlib first, no unrequested abstractions.",
|
||||||
"author": {
|
"author": {
|
||||||
"name": "Dietrich Gebert",
|
"name": "Dietrich Gebert",
|
||||||
|
|||||||
@@ -0,0 +1,20 @@
|
|||||||
|
{
|
||||||
|
"name": "ponytail",
|
||||||
|
"description": "Lazy senior dev mode for AI agents. The best code is the code you never wrote.",
|
||||||
|
"owner": {
|
||||||
|
"name": "Dietrich Gebert",
|
||||||
|
"url": "https://github.com/DietrichGebert"
|
||||||
|
},
|
||||||
|
"plugins": [
|
||||||
|
{
|
||||||
|
"name": "ponytail",
|
||||||
|
"description": "Forces the laziest solution that works. YAGNI, stdlib first, one line over fifty.",
|
||||||
|
"source": "./",
|
||||||
|
"category": "productivity",
|
||||||
|
"tags": ["yagni", "minimalism", "code-review", "productivity"],
|
||||||
|
"commands": "commands/",
|
||||||
|
"skills": "skills/",
|
||||||
|
"hooks": "hooks/copilot-hooks.json"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
@@ -0,0 +1,16 @@
|
|||||||
|
{
|
||||||
|
"name": "ponytail",
|
||||||
|
"description": "Lazy senior dev mode. Forces the simplest, shortest solution that actually works: YAGNI, stdlib first, no unrequested abstractions.",
|
||||||
|
"version": "4.5.0",
|
||||||
|
"author": {
|
||||||
|
"name": "Dietrich Gebert",
|
||||||
|
"url": "https://github.com/DietrichGebert"
|
||||||
|
},
|
||||||
|
"homepage": "https://github.com/DietrichGebert/ponytail",
|
||||||
|
"repository": "https://github.com/DietrichGebert/ponytail",
|
||||||
|
"license": "MIT",
|
||||||
|
"keywords": ["yagni", "minimalism", "code-review", "productivity"],
|
||||||
|
"commands": "commands/",
|
||||||
|
"skills": "skills/",
|
||||||
|
"hooks": "hooks/copilot-hooks.json"
|
||||||
|
}
|
||||||
@@ -0,0 +1,29 @@
|
|||||||
|
name: test
|
||||||
|
|
||||||
|
on:
|
||||||
|
push:
|
||||||
|
branches: [main]
|
||||||
|
pull_request:
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
test:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version: '22'
|
||||||
|
|
||||||
|
- uses: actions/setup-python@v5
|
||||||
|
with:
|
||||||
|
python-version: '3.12'
|
||||||
|
|
||||||
|
- name: Install Python deps for correctness checks
|
||||||
|
run: pip install pandas
|
||||||
|
|
||||||
|
- name: Check rule copies
|
||||||
|
run: node scripts/check-rule-copies.js
|
||||||
|
|
||||||
|
- name: Run tests
|
||||||
|
run: npm test
|
||||||
@@ -14,7 +14,7 @@
|
|||||||
<p align="center">
|
<p align="center">
|
||||||
<img src="https://img.shields.io/github/stars/DietrichGebert/ponytail?style=flat-square&color=111111&label=stars" alt="Stars">
|
<img src="https://img.shields.io/github/stars/DietrichGebert/ponytail?style=flat-square&color=111111&label=stars" alt="Stars">
|
||||||
<img src="https://img.shields.io/github/v/release/DietrichGebert/ponytail?style=flat-square&color=111111&label=release" alt="Release">
|
<img src="https://img.shields.io/github/v/release/DietrichGebert/ponytail?style=flat-square&color=111111&label=release" alt="Release">
|
||||||
<img src="https://img.shields.io/badge/works%20with-11%20agents-111111?style=flat-square" alt="Works with 11 agents">
|
<img src="https://img.shields.io/badge/works%20with-13%20agents-111111?style=flat-square" alt="Works with 13 agents">
|
||||||
<img src="https://img.shields.io/badge/license-MIT-111111?style=flat-square" alt="MIT license">
|
<img src="https://img.shields.io/badge/license-MIT-111111?style=flat-square" alt="MIT license">
|
||||||
</p>
|
</p>
|
||||||
|
|
||||||
@@ -71,6 +71,8 @@ Lazy, not negligent: trust-boundary validation, data-loss handling, security, an
|
|||||||
|
|
||||||
The most effort ponytail will ever ask of you:
|
The most effort ponytail will ever ask of you:
|
||||||
|
|
||||||
|
The Claude Code and Codex plugins run two tiny Node.js lifecycle hooks, so `node` needs to be on your PATH (note for Nix/nvm users: it must be on the non-interactive shell's PATH). If it isn't, the skills still work, the always-on activation just stays quiet instead of erroring on every prompt.
|
||||||
|
|
||||||
### Claude Code
|
### Claude Code
|
||||||
|
|
||||||
```
|
```
|
||||||
@@ -88,6 +90,29 @@ codex
|
|||||||
Open `/plugins`, select the Ponytail marketplace, and install Ponytail. Then
|
Open `/plugins`, select the Ponytail marketplace, and install Ponytail. Then
|
||||||
open `/hooks`, review and trust its two lifecycle hooks, and start a new thread.
|
open `/hooks`, review and trust its two lifecycle hooks, and start a new thread.
|
||||||
|
|
||||||
|
This same install also covers the Codex desktop app: restart the app after installing and it picks up the plugin.
|
||||||
|
|
||||||
|
### GitHub Copilot CLI
|
||||||
|
|
||||||
|
```bash
|
||||||
|
copilot plugin marketplace add DietrichGebert/ponytail
|
||||||
|
copilot plugin install ponytail@ponytail
|
||||||
|
```
|
||||||
|
|
||||||
|
In an interactive Copilot CLI session, use the slash equivalents:
|
||||||
|
|
||||||
|
```
|
||||||
|
/plugin marketplace add DietrichGebert/ponytail
|
||||||
|
/plugin install ponytail@ponytail
|
||||||
|
```
|
||||||
|
|
||||||
|
Copilot CLI namespaces plugin commands by plugin name. For example:
|
||||||
|
|
||||||
|
```text
|
||||||
|
/ponytail:ponytail ultra
|
||||||
|
/ponytail:ponytail-review
|
||||||
|
```
|
||||||
|
|
||||||
### Pi agent harness
|
### Pi agent harness
|
||||||
|
|
||||||
```
|
```
|
||||||
@@ -116,11 +141,11 @@ That was it. He'd be proud. He won't say it.
|
|||||||
|
|
||||||
Active every session, with a handful of commands (see [Commands](#commands)). `/ponytail ultra` exists for when the codebase has wronged you personally. Startup and mode-change text shows the current mode.
|
Active every session, with a handful of commands (see [Commands](#commands)). `/ponytail ultra` exists for when the codebase has wronged you personally. Startup and mode-change text shows the current mode.
|
||||||
|
|
||||||
Cursor, Windsurf, Cline, Copilot, Aider, Kiro: copy the matching rules file from this repo ([`.cursor/rules/`](.cursor/rules/), [`.windsurf/rules/`](.windsurf/rules/), [`.clinerules/`](.clinerules/), [`.github/copilot-instructions.md`](.github/copilot-instructions.md), [`AGENTS.md`](AGENTS.md), [`.kiro/steering/`](.kiro/steering/)).
|
Cursor, Windsurf, Cline, GitHub Copilot (editor), Aider, Kiro: copy the matching rules file from this repo ([`.cursor/rules/`](.cursor/rules/), [`.windsurf/rules/`](.windsurf/rules/), [`.clinerules/`](.clinerules/), [`.github/copilot-instructions.md`](.github/copilot-instructions.md), [`AGENTS.md`](AGENTS.md), [`.kiro/steering/`](.kiro/steering/)).
|
||||||
|
|
||||||
Kiro: copy `.kiro/steering/ponytail.md` to `~/.kiro/steering/` (global) or `.kiro/steering/` in your project.
|
Kiro: copy `.kiro/steering/ponytail.md` to `~/.kiro/steering/` (global) or `.kiro/steering/` in your project.
|
||||||
|
|
||||||
GitHub Copilot CLI: it already reads `AGENTS.md` and `.github/copilot-instructions.md` in a project, or copy the rules into `~/.copilot/copilot-instructions.md` to run ponytail in every project.
|
GitHub Copilot CLI fallback (instruction-only mode): it reads `AGENTS.md` and `.github/copilot-instructions.md` in a project, or copy the rules into `~/.copilot/copilot-instructions.md` to run ponytail in every project. This path keeps always-on guidance, but does not add plugin mode switches or hooks.
|
||||||
|
|
||||||
Antigravity and VS Code with the Codex extension: both read `AGENTS.md`, which this repo ships, so it works from the repo root with no setup (`~/.codex/AGENTS.md` makes Codex global, `.agents/rules/` makes it an always-on rule in Antigravity).
|
Antigravity and VS Code with the Codex extension: both read `AGENTS.md`, which this repo ships, so it works from the repo root with no setup (`~/.codex/AGENTS.md` makes Codex global, `.agents/rules/` makes it an always-on rule in Antigravity).
|
||||||
|
|
||||||
@@ -144,8 +169,11 @@ When changing the compact rule text, keep the agent copies aligned:
|
|||||||
|
|
||||||
```bash
|
```bash
|
||||||
node scripts/check-rule-copies.js
|
node scripts/check-rule-copies.js
|
||||||
|
npm test
|
||||||
```
|
```
|
||||||
|
|
||||||
|
The correctness benchmark spawns Python for email and CSV checks; `python3` is tried before `python`. CSV checks need `pandas` installed locally.
|
||||||
|
|
||||||
## FAQ
|
## FAQ
|
||||||
|
|
||||||
**Does it need a config file?**
|
**Does it need a config file?**
|
||||||
|
|||||||
@@ -38,6 +38,20 @@ function exec(cmd, opts = {}) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// ponytail: probe once at load; macOS and many Linux images ship python3 only.
|
||||||
|
let pythonCmd;
|
||||||
|
function python() {
|
||||||
|
if (pythonCmd) return pythonCmd;
|
||||||
|
for (const cmd of ['python3', 'python']) {
|
||||||
|
if (exec(`${cmd} -c "import sys"`).ok) {
|
||||||
|
pythonCmd = cmd;
|
||||||
|
return pythonCmd;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
pythonCmd = 'python3';
|
||||||
|
return pythonCmd;
|
||||||
|
}
|
||||||
|
|
||||||
// Write content to a temp file, return the path.
|
// Write content to a temp file, return the path.
|
||||||
function tmpFile(ext, content) {
|
function tmpFile(ext, content) {
|
||||||
const p = path.join(os.tmpdir(), `ponytail-bench-${Date.now()}-${Math.random().toString(36).slice(2)}${ext}`);
|
const p = path.join(os.tmpdir(), `ponytail-bench-${Date.now()}-${Math.random().toString(36).slice(2)}${ext}`);
|
||||||
@@ -100,7 +114,7 @@ if failures:
|
|||||||
print("PASS")
|
print("PASS")
|
||||||
`;
|
`;
|
||||||
const f = tmpFile('.py', harness);
|
const f = tmpFile('.py', harness);
|
||||||
const result = exec(`python "${f}"`);
|
const result = exec(`${python()} "${f}"`);
|
||||||
fs.unlinkSync(f);
|
fs.unlinkSync(f);
|
||||||
if (result.ok) return { pass: true, reason: 'Email validator passes all checks' };
|
if (result.ok) return { pass: true, reason: 'Email validator passes all checks' };
|
||||||
return { pass: false, reason: result.stderr || 'Email validator failed' };
|
return { pass: false, reason: result.stderr || 'Email validator failed' };
|
||||||
@@ -194,7 +208,7 @@ else:
|
|||||||
sys.exit(1)
|
sys.exit(1)
|
||||||
`;
|
`;
|
||||||
const f = tmpFile('.py', harness);
|
const f = tmpFile('.py', harness);
|
||||||
const result = exec(`python "${f}"`);
|
const result = exec(`${python()} "${f}"`);
|
||||||
try { fs.unlinkSync(f); } catch (e) {}
|
try { fs.unlinkSync(f); } catch (e) {}
|
||||||
try { fs.unlinkSync(csvPath); } catch (e) {}
|
try { fs.unlinkSync(csvPath); } catch (e) {}
|
||||||
if (result.ok) return { pass: true, reason: 'CSV sum produces correct result (351)' };
|
if (result.ok) return { pass: true, reason: 'CSV sum produces correct result (351)' };
|
||||||
|
|||||||
@@ -17,7 +17,7 @@ to load in a given agent.
|
|||||||
| Windsurf | `.windsurf/rules/ponytail.md` | Project rule. |
|
| Windsurf | `.windsurf/rules/ponytail.md` | Project rule. |
|
||||||
| Cline | `.clinerules/ponytail.md` | Project rule. |
|
| Cline | `.clinerules/ponytail.md` | Project rule. |
|
||||||
| GitHub Copilot | `.github/copilot-instructions.md` | Repository instruction file. |
|
| GitHub Copilot | `.github/copilot-instructions.md` | Repository instruction file. |
|
||||||
| GitHub Copilot CLI | `AGENTS.md`, `.github/copilot-instructions.md`, `~/.copilot/copilot-instructions.md` | Reads custom instructions: per-project from `AGENTS.md` or `.github/copilot-instructions.md`, or globally from `~/.copilot/copilot-instructions.md`. Instruction-tier (no `/ponytail` levels or hooks). |
|
| GitHub Copilot CLI | `.github/plugin/`, `AGENTS.md`, `.github/copilot-instructions.md`, `~/.copilot/copilot-instructions.md` | Plugin-supported (`copilot plugin marketplace add DietrichGebert/ponytail` + `copilot plugin install ponytail@ponytail`). Fallback instruction mode remains: per-project from `AGENTS.md` or `.github/copilot-instructions.md`, or globally from `~/.copilot/copilot-instructions.md` (instruction-tier, no `/ponytail` levels or hooks). |
|
||||||
| Antigravity | `AGENTS.md` | Reads `AGENTS.md` at the repo root as always-on rules (like `.cursorrules`/`CLAUDE.md`); `.agents/rules/` also works for workspace rules. Instruction-tier. |
|
| Antigravity | `AGENTS.md` | Reads `AGENTS.md` at the repo root as always-on rules (like `.cursorrules`/`CLAUDE.md`); `.agents/rules/` also works for workspace rules. Instruction-tier. |
|
||||||
| VS Code + Codex extension | `AGENTS.md` | The Codex extension reads `AGENTS.md` (repo root, or `~/.codex/AGENTS.md` globally). Instruction-tier; the full Codex plugin row above adds `/ponytail` levels and hooks. |
|
| VS Code + Codex extension | `AGENTS.md` | The Codex extension reads `AGENTS.md` (repo root, or `~/.codex/AGENTS.md` globally). Instruction-tier; the full Codex plugin row above adds `/ponytail` levels and hooks. |
|
||||||
| Kiro | `.kiro/steering/ponytail.md` | Steering rule; copy globally or into a project. |
|
| Kiro | `.kiro/steering/ponytail.md` | Steering rule; copy globally or into a project. |
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "ponytail",
|
"name": "ponytail",
|
||||||
"version": "4.4.0",
|
"version": "4.5.0",
|
||||||
"description": "Lazy senior dev mode. Forces the simplest, shortest solution that actually works: YAGNI, stdlib first, no unrequested abstractions.",
|
"description": "Lazy senior dev mode. Forces the simplest, shortest solution that actually works: YAGNI, stdlib first, no unrequested abstractions.",
|
||||||
"contextFileName": "AGENTS.md"
|
"contextFileName": "AGENTS.md"
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,21 @@
|
|||||||
|
{
|
||||||
|
"version": 1,
|
||||||
|
"hooks": {
|
||||||
|
"sessionStart": [
|
||||||
|
{
|
||||||
|
"type": "command",
|
||||||
|
"bash": "node \"${PLUGIN_ROOT}/hooks/ponytail-activate.js\"",
|
||||||
|
"powershell": "node \"${PLUGIN_ROOT}\\hooks\\ponytail-activate.js\"",
|
||||||
|
"timeoutSec": 5
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"userPromptSubmitted": [
|
||||||
|
{
|
||||||
|
"type": "command",
|
||||||
|
"bash": "node \"${PLUGIN_ROOT}/hooks/ponytail-mode-tracker.js\"",
|
||||||
|
"powershell": "node \"${PLUGIN_ROOT}\\hooks\\ponytail-mode-tracker.js\"",
|
||||||
|
"timeoutSec": 5
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
+4
-4
@@ -6,8 +6,8 @@
|
|||||||
"hooks": [
|
"hooks": [
|
||||||
{
|
{
|
||||||
"type": "command",
|
"type": "command",
|
||||||
"command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/ponytail-activate.js\"",
|
"command": "command -v node >/dev/null 2>&1 && node \"${CLAUDE_PLUGIN_ROOT}/hooks/ponytail-activate.js\" || exit 0",
|
||||||
"commandWindows": "node \"$env:CLAUDE_PLUGIN_ROOT\\hooks\\ponytail-activate.js\"",
|
"commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { node \"$env:CLAUDE_PLUGIN_ROOT\\hooks\\ponytail-activate.js\" }",
|
||||||
"timeout": 5,
|
"timeout": 5,
|
||||||
"statusMessage": "Loading ponytail mode..."
|
"statusMessage": "Loading ponytail mode..."
|
||||||
}
|
}
|
||||||
@@ -19,8 +19,8 @@
|
|||||||
"hooks": [
|
"hooks": [
|
||||||
{
|
{
|
||||||
"type": "command",
|
"type": "command",
|
||||||
"command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/ponytail-mode-tracker.js\"",
|
"command": "command -v node >/dev/null 2>&1 && node \"${CLAUDE_PLUGIN_ROOT}/hooks/ponytail-mode-tracker.js\" || exit 0",
|
||||||
"commandWindows": "node \"$env:CLAUDE_PLUGIN_ROOT\\hooks\\ponytail-mode-tracker.js\"",
|
"commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { node \"$env:CLAUDE_PLUGIN_ROOT\\hooks\\ponytail-mode-tracker.js\" }",
|
||||||
"timeout": 5,
|
"timeout": 5,
|
||||||
"statusMessage": "Tracking ponytail mode..."
|
"statusMessage": "Tracking ponytail mode..."
|
||||||
}
|
}
|
||||||
|
|||||||
+25
-13
@@ -2,10 +2,15 @@ const fs = require('fs');
|
|||||||
const path = require('path');
|
const path = require('path');
|
||||||
const { getClaudeDir } = require('./ponytail-config');
|
const { getClaudeDir } = require('./ponytail-config');
|
||||||
|
|
||||||
const isCodex = Boolean(process.env.PLUGIN_DATA);
|
const STATE_FILE = '.ponytail-active';
|
||||||
const statePath = isCodex
|
const isCopilot = Boolean(process.env.COPILOT_PLUGIN_DATA);
|
||||||
? path.join(process.env.PLUGIN_DATA, '.ponytail-active')
|
const isCodex = !isCopilot && Boolean(process.env.PLUGIN_DATA);
|
||||||
: path.join(getClaudeDir(), '.ponytail-active');
|
|
||||||
|
let stateDir = getClaudeDir();
|
||||||
|
if (isCodex) stateDir = process.env.PLUGIN_DATA;
|
||||||
|
if (isCopilot) stateDir = process.env.COPILOT_PLUGIN_DATA;
|
||||||
|
|
||||||
|
const statePath = path.join(stateDir, STATE_FILE);
|
||||||
|
|
||||||
function setMode(mode) {
|
function setMode(mode) {
|
||||||
fs.mkdirSync(path.dirname(statePath), { recursive: true });
|
fs.mkdirSync(path.dirname(statePath), { recursive: true });
|
||||||
@@ -17,23 +22,30 @@ function clearMode() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
function writeHookOutput(event, mode, context = '') {
|
function writeHookOutput(event, mode, context = '') {
|
||||||
if (!isCodex) {
|
if (isCopilot) {
|
||||||
process.stdout.write(context);
|
// Copilot reads additionalContext on SessionStart; ignores output elsewhere.
|
||||||
|
process.stdout.write(JSON.stringify(
|
||||||
|
event === 'SessionStart' && context ? { additionalContext: context } : {}));
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
const output = { systemMessage: `PONYTAIL:${mode.toUpperCase()}` };
|
if (isCodex) {
|
||||||
if (context) {
|
const output = { systemMessage: `PONYTAIL:${mode.toUpperCase()}` };
|
||||||
output.hookSpecificOutput = {
|
if (context) {
|
||||||
hookEventName: event,
|
output.hookSpecificOutput = {
|
||||||
additionalContext: context,
|
hookEventName: event,
|
||||||
};
|
additionalContext: context,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
process.stdout.write(JSON.stringify(output));
|
||||||
|
return;
|
||||||
}
|
}
|
||||||
process.stdout.write(JSON.stringify(output));
|
process.stdout.write(context);
|
||||||
}
|
}
|
||||||
|
|
||||||
module.exports = {
|
module.exports = {
|
||||||
clearMode,
|
clearMode,
|
||||||
isCodex,
|
isCodex,
|
||||||
|
isCopilot,
|
||||||
setMode,
|
setMode,
|
||||||
writeHookOutput,
|
writeHookOutput,
|
||||||
};
|
};
|
||||||
|
|||||||
@@ -4,6 +4,9 @@
|
|||||||
"description": "Lazy senior dev mode for AI agents. The best code is the code you never wrote.",
|
"description": "Lazy senior dev mode for AI agents. The best code is the code you never wrote.",
|
||||||
"keywords": ["pi-package", "pi", "skills", "ponytail"],
|
"keywords": ["pi-package", "pi", "skills", "ponytail"],
|
||||||
"license": "MIT",
|
"license": "MIT",
|
||||||
|
"scripts": {
|
||||||
|
"test": "node --test tests/*.test.js && npm test --prefix pi-extension"
|
||||||
|
},
|
||||||
"pi": {
|
"pi": {
|
||||||
"extensions": ["./pi-extension/index.js"],
|
"extensions": ["./pi-extension/index.js"],
|
||||||
"skills": ["./skills"]
|
"skills": ["./skills"]
|
||||||
|
|||||||
@@ -0,0 +1,33 @@
|
|||||||
|
#!/usr/bin/env node
|
||||||
|
// Smoke test for the Copilot plugin adapter: keep command wiring minimal and
|
||||||
|
// ensure the debt command is part of the shared command surface.
|
||||||
|
|
||||||
|
const test = require('node:test');
|
||||||
|
const assert = require('node:assert/strict');
|
||||||
|
const fs = require('fs');
|
||||||
|
const path = require('path');
|
||||||
|
|
||||||
|
const root = path.join(__dirname, '..');
|
||||||
|
const REQUIRED_COMMAND_FILES = [
|
||||||
|
'ponytail.toml',
|
||||||
|
'ponytail-review.toml',
|
||||||
|
'ponytail-audit.toml',
|
||||||
|
'ponytail-debt.toml',
|
||||||
|
];
|
||||||
|
|
||||||
|
function readJSON(relPath) {
|
||||||
|
return JSON.parse(fs.readFileSync(path.join(root, relPath), 'utf8'));
|
||||||
|
}
|
||||||
|
|
||||||
|
test('copilot plugin command directory includes ponytail-debt', () => {
|
||||||
|
const manifest = readJSON('.github/plugin/plugin.json');
|
||||||
|
assert.equal(manifest.name, 'ponytail');
|
||||||
|
assert.equal(manifest.commands, 'commands/');
|
||||||
|
|
||||||
|
for (const file of REQUIRED_COMMAND_FILES) {
|
||||||
|
assert.ok(
|
||||||
|
fs.existsSync(path.join(root, manifest.commands, file)),
|
||||||
|
`missing command file: ${manifest.commands}${file}`,
|
||||||
|
);
|
||||||
|
}
|
||||||
|
});
|
||||||
@@ -16,6 +16,12 @@ const MANIFEST = 'gemini-extension.json';
|
|||||||
const EXTENSION_NAME = 'ponytail';
|
const EXTENSION_NAME = 'ponytail';
|
||||||
// Floating refs are a supply-chain footgun; the manifest version must be pinned.
|
// Floating refs are a supply-chain footgun; the manifest version must be pinned.
|
||||||
const PINNED_SEMVER = /^\d+\.\d+\.\d+$/;
|
const PINNED_SEMVER = /^\d+\.\d+\.\d+$/;
|
||||||
|
const VERSIONED_MANIFESTS = [
|
||||||
|
'gemini-extension.json',
|
||||||
|
'.claude-plugin/plugin.json',
|
||||||
|
'.codex-plugin/plugin.json',
|
||||||
|
'.github/plugin/plugin.json',
|
||||||
|
];
|
||||||
// Gemini auto-discovers these by directory; the manifest is only useful if they exist.
|
// Gemini auto-discovers these by directory; the manifest is only useful if they exist.
|
||||||
const REUSED_COMMANDS = ['commands/ponytail.toml', 'commands/ponytail-review.toml'];
|
const REUSED_COMMANDS = ['commands/ponytail.toml', 'commands/ponytail-review.toml'];
|
||||||
const REUSED_SKILLS = ['skills/ponytail/SKILL.md'];
|
const REUSED_SKILLS = ['skills/ponytail/SKILL.md'];
|
||||||
@@ -46,9 +52,15 @@ test('manifest names the ponytail extension with a pinned version', () => {
|
|||||||
});
|
});
|
||||||
|
|
||||||
test('version stays aligned with the other plugin manifests', () => {
|
test('version stays aligned with the other plugin manifests', () => {
|
||||||
const manifest = loadManifest();
|
const versions = VERSIONED_MANIFESTS.map((rel) => {
|
||||||
const claude = JSON.parse(read('.claude-plugin/plugin.json'));
|
const manifest = JSON.parse(read(rel));
|
||||||
assert.equal(manifest.version, claude.version);
|
assert.match(manifest.version, PINNED_SEMVER, `${rel} version must be pinned semver`);
|
||||||
|
return manifest.version;
|
||||||
|
});
|
||||||
|
const [sharedVersion, ...rest] = versions;
|
||||||
|
for (const version of rest) {
|
||||||
|
assert.equal(version, sharedVersion);
|
||||||
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
test('contextFileName resolves to a file carrying the ponytail rules', () => {
|
test('contextFileName resolves to a file carrying the ponytail rules', () => {
|
||||||
|
|||||||
@@ -99,5 +99,44 @@ assert.equal(
|
|||||||
'flag must not land in ~/.claude when CLAUDE_CONFIG_DIR is set',
|
'flag must not land in ~/.claude when CLAUDE_CONFIG_DIR is set',
|
||||||
);
|
);
|
||||||
|
|
||||||
|
const copilotData = path.join(temp, 'copilot-data');
|
||||||
|
const codexData = path.join(temp, 'codex-data-shadow');
|
||||||
|
result = run('ponytail-activate.js', {
|
||||||
|
HOME: home,
|
||||||
|
USERPROFILE: home,
|
||||||
|
COPILOT_PLUGIN_DATA: copilotData,
|
||||||
|
PLUGIN_DATA: codexData,
|
||||||
|
PONYTAIL_DEFAULT_MODE: 'full',
|
||||||
|
});
|
||||||
|
assert.equal(result.status, 0, result.stderr);
|
||||||
|
assert.equal(fs.readFileSync(path.join(copilotData, '.ponytail-active'), 'utf8'), 'full');
|
||||||
|
assert.equal(
|
||||||
|
fs.existsSync(path.join(codexData, '.ponytail-active')),
|
||||||
|
false,
|
||||||
|
'copilot hooks must not write mode state to codex PLUGIN_DATA',
|
||||||
|
);
|
||||||
|
output = JSON.parse(result.stdout);
|
||||||
|
assert.match(output.additionalContext, /PONYTAIL MODE ACTIVE — level: full/);
|
||||||
|
|
||||||
|
result = run(
|
||||||
|
'ponytail-mode-tracker.js',
|
||||||
|
{
|
||||||
|
HOME: home,
|
||||||
|
USERPROFILE: home,
|
||||||
|
COPILOT_PLUGIN_DATA: copilotData,
|
||||||
|
PLUGIN_DATA: codexData,
|
||||||
|
},
|
||||||
|
JSON.stringify({ prompt: '/ponytail ultra' }),
|
||||||
|
);
|
||||||
|
assert.equal(result.status, 0, result.stderr);
|
||||||
|
assert.equal(fs.readFileSync(path.join(copilotData, '.ponytail-active'), 'utf8'), 'ultra');
|
||||||
|
assert.equal(
|
||||||
|
fs.existsSync(path.join(codexData, '.ponytail-active')),
|
||||||
|
false,
|
||||||
|
'copilot mode tracker must keep codex PLUGIN_DATA untouched',
|
||||||
|
);
|
||||||
|
output = JSON.parse(result.stdout);
|
||||||
|
assert.deepEqual(output, {});
|
||||||
|
|
||||||
fs.rmSync(temp, { recursive: true, force: true });
|
fs.rmSync(temp, { recursive: true, force: true });
|
||||||
console.log('hook compatibility checks passed');
|
console.log('hook compatibility checks passed');
|
||||||
|
|||||||
Reference in New Issue
Block a user