chore: use $DOC_ROOT for the requests path and drop the consumer-project reference - #65
Closed
poxet wants to merge 1 commit into
Closed
chore: use $DOC_ROOT for the requests path and drop the consumer-project reference#65poxet wants to merge 1 commit into
poxet wants to merge 1 commit into
Conversation
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.
Documentation-only cleanup of
.claude/mission.md, completing the ecosystem-wide pass across all 16 Tharga projects. The rules behind it live inshared-instructions.md→ Feature Requests (cross-project) → External consumers.Overlaps with #64 — read before merging
#64 already contains this same fix (commit
a07297d, "use DOC_ROOT for requests path and drop the consumer-repo reference"), bundled with a version-line bump and comment cleanup.This PR exists so Cache is covered on its own branch like every other project, and so the fix is not gated on the rest of #64. Whichever merges first wins:
.claude/mission.mdhunk will conflict or no-op; drop that hunk from chore: version line to 1.1, comment cleanup, and mission.md request sources #64 and keep the rest.Both make the same two edits, so there is no risk of them disagreeing about the outcome — only about which commit carries it.
The two changes
1. The requests path was a literal machine path.
A hard-coded path resolves on exactly one machine and silently finds nothing everywhere else — and "silently finds nothing" is indistinguishable from "there was nothing pending".
2. Removed the reference to a consuming product's checkout.
- **Eplicta requests**: `$DEV_ROOT/Eplicta/plan/requests.md` — check for requests from Eplicta on startupA Tharga project should not know about a specific consuming product. The line made this project's startup depend on a checkout it has no claim to, and gave one consumer an inbound route nobody else could use.
The boundary now runs one way. A consuming product raises a GitHub issue on this repository — that is the whole inbound channel. Outbound is a comment on that issue and nothing further: no follow-up entry, no adoption tracker, no writing into their files. Whoever asked tracks their own adoption.
Nothing is lost: open GitHub issues are already a mandatory request source on every startup, independent of what
mission.mdlists.Note on merging
Merging triggers the full CI pipeline including
release, because no repo filters by path — so it publishes a new patch version identical in content to the current one. That cost applies once, whichever of the two PRs carries the change.