Record fleet-diff as shipped and correct its contract in the process brainstorm #44
No reviewers
Labels
No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
allod/strategy!44
Loading…
Reference in a new issue
No description provided.
Delete branch "agent/fleet-diff-shipped"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Corrects one paragraph in a brainstorm that the shipped tool has overtaken.
No machine changes and no agent behaviour changes — this repo is Markdown only. What changes is that a reader of the development-process brainstorm is no longer told to build a tool that exists, with a contract it does not have. What stays identical: the proposal itself, which is still unadopted, and every other section of the document. What proves it: one paragraph diff, no other file touched. If it is wrong, revert the commit; nothing depends on it.
Why this and not an archive
README.mdsays a plan moves toarchive/once implemented or superseded. That is the wrong move here. The document is a project-direction proposal for a land-inert and gate-activation development process, and that process is not adopted —memory'sgit-workflow.mdhas no such section. Only the tools paragraph is overtaken, so only the tools paragraph changes.What was wrong
The paragraph specified "non-zero exit when anything changes so it composes with expectations". The tool that shipped in allod/tools#136 deliberately does the opposite: an expectation is declared with
--expector--expect-none, a mismatch fails in both directions, and a run with no expectation is report-only and exits 0, because a run that asserts nothing should not gate anything. That reasoning is recorded on the tool's own issue, which supersedes this line on purpose.Two smaller staleness items in the same paragraph: overrides now take a revision rather than a URL, since the checkout's lock already records which repository each input came from; and the sizing estimate ("roughly the size of
flake-status, one PR, R2, one review pass") described work that has since happened, so it is dropped rather than left reading as outstanding.The paragraph now says plainly that the section is a record of what was proposed rather than work outstanding, and points at
flake/README.mdinallod/toolsas the reference.Risk
R0. Documentation in a planning repo, one paragraph, no consumer.
Validation
git diff --statshows one file and five inserted lines against one deleted. No other file in the repo mentionsfleet-diff— checked with a repo-wide grep before editing, which is also how this paragraph was found.Deliberately not recorded here: the limit that a fleet diff cannot distinguish a real change from a store-path move for an input whose path is embedded in a machine's configuration. That finding belongs with its cause, which is allod/archetypes#21 and the
allod-toolsinterpolation beside it, not with the document that observes it.View command line instructions
Checkout
From your project repository, check out a new branch and test the changes.Merge
Merge the changes and update on Forgejo.Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.