Repository navigation
COMMON.md: run the engine's presence.ps1 -Fleet, now that the engine copy has it (vault BACKLOG #3674) - #229
Merged
Merged
Conversation
…KLOG #3674, step 4) Engine PR 2355 gave the engine's scripts/coord/presence.ps1 a -Fleet switch. The exceptions row drops the korus by-path entry, the by-path instruction from korus PR 228 now names the engine copy, and the caveat describes the engine behaviour: it always scans the caller's repository, and a -Fleet roster still cannot list sessions whose cwd is outside git, gone with no checkout above it, or on another machine. Proposed PR title: COMMON.md runs the engine's presence.ps1 -Fleet (vault #3674 step 4) Proposed ledger banner: #3674 step 4 built (korus COMMON.md and wiki note); findings 4 to 6 of the pass F note remain.
…CKLOG #3674, step 4) The instruction requires an engine worktree current with origin/main, cites the PR 2355 merge commit, warns off the korus copy, qualifies the always-scanned claim, and says the engine copy exits 0 when the roster is incomplete. The 2026-09-19 measurement names the korus copy.
…p 4) The command names the engine worktree explicitly, the fence shapes hold for every scanned repository, the exit-0 note covers the not-in-git path, and the wiki line says the vault page still names the korus route.
Collaborator
Author
|
Lander: merging at pinned head 54fa1f3. The Lander's xhigh review covered the whole diff at 570cd84, including the round-1 fixes. The fix commit 54fa1f3 applies that review's four fixes; the Lander checked its diff, which adds two paragraph splits for the prose ratchets. Required gates are green. Follow-ups are filed in vault (#3752, via vault PR 2500). |
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.
Vault BACKLOG #3674, closing step 4. Engine PR MEFORORG/MessageFoundry#2355 (merged as
45f0346d4) gave the engine'sscripts/coord/presence.ps1a-Fleetswitch. So readers no longer run korus's copy by path for it. This changes onlyroles/COMMON.md(+20/-6).Changes
presence.ps1 -Fleetentry, since the engine copy now takes-Fleet.presence.ps1 -Fleetnow reads true. It scans your repo and every repo the records name, on every account. It cannot see another machine, nor at least the cwd shapes listed below.origin/main. The source was read at engineorigin/main064700f2don 2026-10-17. The new text:-Fleetroster without a seat does not prove the seat absent.Roster UNAVAILABLE/INCOMPLETEline or the-Jsonreceipt, not the exit code.occupancy.ps1shapes. Another repo named only by a gone cwd is not scanned.The wiki note
coord/presence/fleet-flagis corrected separately, by event20261017T144420738Z-zp60qd. It corrects20261012T152837537Z-l477qc.Checks
test_prose_rules_hold,test_role_cards,test_the_seat_that_opens_a_pr_is_stated_onceandtest_presence_states_its_scope: 107 passed, 1 failed. The failure istest_the_default_run_prints_the_scope_warning. It fails the same way onorigin/main6b9df03, and it runs korus's script, not COMMON.md.merge-tree --write-tree origin/main origin/fleet-flag-engine-copyexits 0.QA
Open:
scripts/coord/presence.ps1still ships the weaker-Fleet, and its default-run warning points at itself. COMMON.md now warns readers off it. Changing or retiring the script is a new row.570cd84were not re-reviewed.The engine copy exits 0 on an incomplete roster, while korus's exits 2. That is deliberate: the engine's existing tests assert exit 0, and the PR 2355 brief kept it. COMMON.md now says to read the line, not the exit code. So it is not a finding.
Ledger banner (proposed)
#3674 step 4 done: korus COMMON.md points
-Fleetat the engine copy and the wiki note is corrected; all four closing steps are met.