docs(fork): document .sh shim delegation strategy (#69) - #124
Merged
Conversation
Counter-position to upstream MemPalace#1069. Upstream wants to consolidate the per-event hook .sh wrappers into shims that delegate to `mempalace hook run`. This fork went the opposite direction post-2026-05-11: shims delegate to `palace-daemon/clients/hook.py`, which talks HTTP to a single FastAPI gateway — `mempalace` is no longer in the hook call path. docs/fork-decisions/sh-shim-strategy.md captures: - why the .sh shims stay (back-compat, ops, graceful absence) - why delegation goes through palace-daemon (single-writer gateway) - the delegation pattern diagram - a copy-paste template - re-converge conditions scripts/mempalace-search.sh is a non-hook sample of the same pattern (HTTP → /search), for contributors adding new delegating shims. Refs #69
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
2 tasks
Closed
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes/tracks techempower-org/mempalace#69. Documents the fork's counter-position to upstream MemPalace/mempalace#1069:
hooks/mempal_*_hook.shinto thin wrappers delegating tomempalace hook runMemPalace/mempalace#1069 direction: consolidate per-event.shhook wrappers into shims delegating tomempalace hook run(Python CLI in themempalacepackage).palace-daemon/clients/hook.py(stdlib-only Python) which speaks HTTP to a single FastAPI gateway.mempalaceis no longer in the hook call path.What's in here
docs/fork-decisions/sh-shim-strategy.md— the why (back-compat + ops + graceful absence), the delegation diagram, the shim template, and re-converge conditions.scripts/mempalace-search.sh— non-hook sample of the same delegation pattern (HTTP GET to/search), as a copy-paste template for contributors adding new delegating shims.docs/fork-changes.yaml—sh-shim-strategy-docentry pointing at the docs commit.FORK_CHANGELOG.md— regenerated viascripts/render-docs.py.No runtime change. The shims this documents (
.claude-plugin/hooks/*.sh,.codex-plugin/hooks/*.sh) have been delegating to palace-daemon since the 2026-05-11 split-brain fix; this PR just records the decision.Test plan
bash scripts/check-docs.shpasses (4/4 clean — commit hashes resolve, FORK_CHANGELOG matches YAML, PR-state claims match upstream).scripts/render-docs.py --checkexit 0 (rendered docs are idempotent against the YAML).mempal-stop-hook.sh,mempal-precompact-hook.sh,.codex-plugin/hooks/mempal-hook.sh).scripts/mempalace-search.shsyntactically clean (bash -n); intentionally not wired to CI — it's a template, not a feature.🤖 Generated with Claude Code