Commit Graph

23 Commits

Author SHA1 Message Date
alvins82 d302797c51 fix(hooks)!: harden aliased Pi and OMP extension lifecycle
Treat pre-existing extensions without ownership state as uncertain, cover project-scope aliases, clean canonical sidecars after symlink removal, and classify protected extension overwrites as breaking.

BREAKING CHANGE: non-interactive installs of modified or unrelated Pi/OMP extensions now require --auto-patch to approve overwrites.
2026-09-03 11:04:19 +12:00
alvins82 ba6a1f57a3 fix: harden shared extension ownership handling 2026-09-03 09:35:36 +12:00
alvins82 8c25ada388 fix: harden shared extension ownership 2026-08-29 13:22:28 +12:00
alvins82 42a91474e9 fix: track shared agent ownership safely 2026-08-29 11:29:50 +12:00
alvins82 48ee59ec1b fix: address latest OMP review feedback 2026-08-29 09:08:48 +12:00
alvins82 256d4a77bb fix: handle extension safety review feedback 2026-08-28 13:53:15 +12:00
alvins82 002248839e fix: address second OMP review feedback 2026-08-28 13:07:56 +12:00
alvins82 d268e8f282 fix: address follow-up review feedback 2026-08-28 12:11:51 +12:00
alvins82 12a7cf0946 Merge remote-tracking branch 'origin/develop' into omp-shared
# Conflicts:
#	hooks/README.md
2026-08-27 12:45:46 +12:00
alvins82 ad51059674 fix: address OMP integration review feedback 2026-08-27 12:36:38 +12:00
Nicolas Le Cam 322db4da98 Merge pull request #2938 from raj921/fix/pi-slow-startup
perf(hook): avoid Pi package barrel import
2026-08-27 01:29:58 +02:00
Nicolas Le Cam fb61481a76 address review comments: explain type-guard rationale and link #2753
Also fixes two docs left stale by the barrel-import removal: hooks/pi/README.md
and hooks/README.md still described bash-tool detection via the removed
isToolCallEventType value import.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2026-08-27 01:14:46 +02:00
alvins82 4c72946868 feat(pi): shared extension gains guarded OMP status/label hooks
The extension is shared with OMP via its legacy-pi-compat layer. Add
guarded helpers that are strict no-ops on Pi: a persistent
"RTK disabled: <reason>" session status registered on session_start
(OMP wipes one-shot notify toasts on the initial render) and a
setLabel("RTK") UI label set before the version probe, so the
extension is identifiable in the session UI even when rtk is missing
or too old.

Co-authored-by: makoMakoGo <makoMakoGo@users.noreply.github.com>
2026-08-25 19:04:07 +12:00
Adrien Eppling a1673f7428 docs: scope savings claims to bash output and document the estimator
RTK was documented as delivering "60-90% token savings", which reads as a
cost reduction. What RTK actually reduces is bash output bytes. Those are
one contributor to input tokens, which are themselves only part of a bill
that also counts output tokens, so the reduction dilutes at every step.

- add docs/guide/resources/savings-explained.md as the canonical explainer:
  the savings chain, both estimators, and what RTK does not reduce
- rescope the headline claim across README (7 languages), the guide, hook
  rules, agent definitions and module READMEs
- relabel per-command tables as bash output reduction, keeping every figure
- document that reported tokens are estimates: rtk gain uses bytes/4
  (src/core/tracking.rs), filter tests use split_whitespace().count().
  Neither is a real tokenizer, so ratios hold but absolute counts do not

Remove figures that had no source: the $3/Mtok constant and its $36
example, the +/-10% tokenization accuracy claim, the 99.5% hook-install
figure, the invented session tables in README and INSTALL, and the 30-50%
parser range.

CHANGELOG is untouched. Shipped release notes stay as a historical record.
2026-07-22 18:33:54 +02:00
raj921 3e7d02d7b4 perf(hook): avoid Pi package barrel import 2026-07-11 01:05:51 +05:30
gitbluf f6a54518bd chore: minor cleanup 2026-05-23 01:58:47 +02:00
gitbluf 8bc8b46384 chore: sync with pi api 2026-05-23 01:57:37 +02:00
gitbluf 9bd6e6f392 feat: address pr suggestions 2026-05-23 01:36:07 +02:00
gitbluf cb1661e68d feat(init): remove --pi flag, canonicalize Pi install to --agent pi
chore: sync the codebase after mergew
2026-05-11 21:46:46 +02:00
Marko Petrovic a7cab79084 chore: update hooks/pi/rtk.ts to reflect new package name
Co-authored-by: Jean du Plessis <jeandp@gmail.com>
2026-05-08 00:32:31 +02:00
gitbluf b2a3ad9443 feat: rm rtk awareness injection 2026-05-06 18:44:27 +02:00
gitbluf d6e27527ea refactor: handling of uninstallation 2026-05-06 16:38:01 +02:00
gitbluf 1da5793b92 feat(hooks): add Pi coding agent integration 2026-05-06 15:43:04 +02:00