Skip to content

feat(hooks): add Pi extension target#2322

Closed
RaviTharuma wants to merge 1 commit into
dyoshikawa:mainfrom
RaviTharuma:feat/pi-hooks-2319
Closed

feat(hooks): add Pi extension target#2322
RaviTharuma wants to merge 1 commit into
dyoshikawa:mainfrom
RaviTharuma:feat/pi-hooks-2319

Conversation

@RaviTharuma

Copy link
Copy Markdown

Summary

  • add export-only Pi hook support through one RuleSync-owned generated extension
  • generate project .pi/extensions/rulesync-hooks.ts or global .pi/agent/extensions/rulesync-hooks.ts
  • map conservative lifecycle/tool events, run command hooks with JSON stdin, support matchers and timeouts
  • protect deletion with an ownership marker and leave unrelated Pi extensions untouched
  • document that import and blocking/mutation semantics are not claimed

Test plan

  • pnpm check
  • focused Pi/processor tests
  • pnpm test — 324 files, 7,276 tests passed

Closes #2319.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

2 similar comments
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@RaviTharuma

Copy link
Copy Markdown
Author

Closing temporarily to comply with this repository’s limit of two concurrent open PRs for external contributors. The implementation remains available on RaviTharuma:feat/pi-hooks-2319 and is linked from #2319. I will reopen this PR after one of #2320 or #2321 is merged or closed.

@RaviTharuma

Copy link
Copy Markdown
Author

Closing as superseded by maintainer PR #2324, which implemented Pi hooks support and was merged into main. Thank you for picking this up.

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.

Add Pi Coding Agent hooks support via generated extension

1 participant