docs: setup-instance step 15 names --no-push for a local-only first publish; gates.md and a docstring follow #159
Files changed: - CHANGES.md - VERSION - instructions/gates.md - instructions/setup-instance.md - tools/chemenu/commands/git_publish.py
This commit is contained in:
1 parent
61130ce55d
commit
03743ebbc0
5 files changed
+19
-5
No files matched your search
+12
-1
@@ -59,7 +59,7 @@ concern - readable here, never shipped as something to parse.
|
|||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
## 8.0.0-beta.10 - 2026-09-30 - publish: the gate lists the staged state; a missing or unreachable remote stops before the commit
|
## 8.0.0-beta.11 - 2026-09-30 - setup-instance step 15 names --no-push for a local-only first publish; gates.md and a docstring follow #159
|
||||||
|
|
||||||
**Author:** Torben Nehmer
|
**Author:** Torben Nehmer
|
||||||
|
|
||||||
@@ -117,8 +117,19 @@ concern - readable here, never shipped as something to parse.
|
|||||||
- bug-report instruction: offer WIKI_TRACE=1 for a reproduction
|
- bug-report instruction: offer WIKI_TRACE=1 for a reproduction
|
||||||
- reports/CONTRACT.md: only the collector's two counting calls take the bugreport session id
|
- reports/CONTRACT.md: only the collector's two counting calls take the bugreport session id
|
||||||
- bug-report instruction: step 1 no longer calls every bundle unpseudonymised
|
- bug-report instruction: step 1 no longer calls every bundle unpseudonymised
|
||||||
|
- setup-instance step 15 names --no-push for a local-only first publish; gates.md and a docstring follow #159
|
||||||
<!-- /wikitool:bumps -->
|
<!-- /wikitool:bumps -->
|
||||||
|
|
||||||
|
### setup-instance step 15 names --no-push for a local-only first publish; gates.md and a docstring follow #159
|
||||||
|
|
||||||
|
The close-out review of #159 found three places the change had not reached. `setup-instance.md`
|
||||||
|
step 15 shows the first `publish` without `--no-push` and announces exit 42; a local-only instance
|
||||||
|
now gets exit 1 there, before the gate, so the step says so and names the flag. `gates.md` said
|
||||||
|
the Mass-Update Gate counts "working-tree changes before `publish` stages them", which no longer
|
||||||
|
describes a gate that stages into a scratch index; it now says the uncommitted changes `publish` is
|
||||||
|
about to stage. The `_reconcile_summary` docstring still named "no remote" as an outcome with no
|
||||||
|
message.
|
||||||
|
|
||||||
### publish: the gate lists the staged state; a missing or unreachable remote stops before the commit (Gitea #159)
|
### publish: the gate lists the staged state; a missing or unreachable remote stops before the commit (Gitea #159)
|
||||||
|
|
||||||
**The Mass-Update Gate counted a path twice.** `collect_changes` read `git status --porcelain`, which
|
**The Mass-Update Gate counted a path twice.** `collect_changes` read `git status --porcelain`, which
|
||||||
|
|||||||
@@ -139,8 +139,8 @@ deletes the material and keeps only the reason and a sha256 in `mcp-upload/ledge
|
|||||||
|
|
||||||
`upstream merge` (a private instance taking a stack update - see
|
`upstream merge` (a private instance taking a stack update - see
|
||||||
[private-instance.md](private-instance.md)) can update or delete dozens of stack-owned paths in
|
[private-instance.md](private-instance.md)) can update or delete dozens of stack-owned paths in
|
||||||
one commit, and the Mass-Update Gate does not see any of it. The gate counts *working-tree*
|
one commit, and the Mass-Update Gate does not see any of it. The gate counts the *uncommitted*
|
||||||
changes before `publish` stages them; by the time `upstream merge` commits, the change is
|
changes `publish` is about to stage; by the time `upstream merge` commits, the change is
|
||||||
already history, and the commit it made is not what a later `publish` would be staging - that
|
already history, and the commit it made is not what a later `publish` would be staging - that
|
||||||
publish sees only whatever this session adds on top. A merge touching 200 files therefore goes
|
publish sees only whatever this session adds on top. A merge touching 200 files therefore goes
|
||||||
out ungated the moment it is pushed.
|
out ungated the moment it is pushed.
|
||||||
|
|||||||
@@ -302,6 +302,9 @@ and ready for its first ingest.
|
|||||||
`--confirm <token>` line that publishes once they approve. Details on the gate:
|
`--confirm <token>` line that publishes once they approve. Details on the gate:
|
||||||
[gates.md](gates.md).
|
[gates.md](gates.md).
|
||||||
|
|
||||||
|
A local-only instance (step 4) adds `--no-push` here too. Without it the call ends with exit
|
||||||
|
1 before the gate, because there is no remote to publish to, and commits nothing.
|
||||||
|
|
||||||
16. **Restart the agent session.** Harnesses read the skill directories at startup; only
|
16. **Restart the agent session.** Harnesses read the skill directories at startup; only
|
||||||
afterwards are `wiki-ingest`, `wiki-query`, `wiki-manage`, `wiki-lint`, `wiki-status` and
|
afterwards are `wiki-ingest`, `wiki-query`, `wiki-manage`, `wiki-lint`, `wiki-status` and
|
||||||
`gtd-weekly-review` available.
|
`gtd-weekly-review` available.
|
||||||
|
|||||||
@@ -914,7 +914,7 @@ def rebase_review_message(outcome: ReconcileOutcome, remote: str, branch: str, c
|
|||||||
|
|
||||||
def _reconcile_summary(outcome: ReconcileOutcome, remote: str, branch: str) -> str:
|
def _reconcile_summary(outcome: ReconcileOutcome, remote: str, branch: str) -> str:
|
||||||
"""One line for the outcomes that do not fail or gate - what to tell the caller, or "" for
|
"""One line for the outcomes that do not fail or gate - what to tell the caller, or "" for
|
||||||
the ones not worth narrating every time (`up-to-date`, no remote)."""
|
`conflict` and `needs-review`, which `apply_reconcile` raises on instead."""
|
||||||
if outcome.status == "fast-forwarded":
|
if outcome.status == "fast-forwarded":
|
||||||
return f"Pulled {len(outcome.pulled_commits)} commit(s) from {remote}/{branch}."
|
return f"Pulled {len(outcome.pulled_commits)} commit(s) from {remote}/{branch}."
|
||||||
if outcome.status == "local-ahead":
|
if outcome.status == "local-ahead":
|
||||||
|
|||||||
Reference in new issue
Block a user