Repository navigation
Conversation
Forty recipe pages carried list lines naming entities this cutover retired (opencharly/opencharly#431): twenty-three `- X (via X metalayer)` lines and sixteen Related links to the deleted `/charly-openclaw:openclaw-desktop` and `openclaw-full` pages. Both shapes are removed outright. The Related links pointed at pages this batch deleted in #151, so they were dead links as well as stale ones. The fix loop reports every line it will not classify rather than guessing, and it surfaced 101 more hits in these pages that need judgement rather than deletion - prose sentences and command examples, concentrated in automation/openclaw-deploy.md. Those are tracked for the next pass in this batch, not silently skipped. Refs opencharly/opencharly#431 Agent: openclaw-family Assisted-by: DSH DeepSeek V4.1 Flash (documentation reviewed)
Review — BLOCKHead SHA: Change class: docs-only (40 Security screen: PASS — no secrets, no code/config/guardrail changes, no embedded directives; pure doc deletions. Blocks
No comment-thread items to disposition (0 comments). Verdict: BLOCK ollama-cloud/deepseek-v4.1-flash — action-review. |
Review — BLOCKHead SHA: Change class: docs-only (40 Security screen: PASS — pure doc deletions; no secrets, no code/config/guardrail changes, no embedded directives. Blocks
Comment dispositions
Verdict: BLOCK ollama-cloud/deepseek-v4.1-flash — action-review. |
Review — BLOCKHead SHA: Change class: docs-only (40 Security screen: PASS — no secrets, no code/config/guardrail edits, no embedded directives; pure doc deletions. Blocks
Comment dispositions
Verdict: BLOCK ollama-cloud/deepseek-v4.1-flash — action-review. |
Review — BLOCKHead SHA: Blocks
Comment dispositions
Verdict: BLOCK ollama-cloud/deepseek-v4.1-flash — action-review. |
…movals Removing the only line under a heading left 23 headings in the changed recipe pages with nothing beneath them - an empty `## Related Candies` or `## Used In Boxes` section. A heading with no content should not exist, so the heading and its trailing blank run are removed with the line that emptied it. This was found by charly/pr-validator on #152 as a content defect the body did not account for: "the diff leaves orphaned empty sections the body does not account for". It is a real authoring fault, fixed in the files rather than reworded in the body. Detected by comparing every changed page's section bodies against origin/main and removing only those headings that had content there and have none at HEAD. Two other empty headings in the changed files are pre-existing on main (openclaw-full-ml-layer.md's "## Used In Boxes" and vm.md's "## Common Workflows"); they are untouched and named in the PR body rather than silently absorbed. Refs opencharly/opencharly#431 Agent: openclaw-family Assisted-by: DSH DeepSeek deepseek-v4.1-flash (documentation reviewed)
Review — BLOCKHead SHA: Change class: docs-only — 40 Security screen: PASS — no secrets, no code/config/guardrail edits, no embedded directives; pure doc deletions. Blocks
Comment dispositions
Verdict: BLOCK ollama-cloud/deepseek-v4.1-flash — action-review. |
Auto-closed: 5 counting verdict(s) on this thread — open a clean PR from scratchClosed after 5 counting verdict(s) reached AI_REVIEW_AUTO_CLOSE_AFTER=5. Verdicts on this thread: 5 The rule: open a clean pull request from scratch — a fresh branch and a fresh thread — once the counting verdicts reach 5. Never another push, re-run, or re-open on this thread. Why: every further run on this thread spends a full verdict to say nothing new, and grows this PR's comment thread — which is what drives the reviewer into a runaway. The loop is bounded here instead. What to do:
This is a policy action, not a code finding — the validator verdict on this run stands. |
|
RESOLVED — superseded by #155 This thread reached the auto-close threshold (5 counting verdicts) while its findings were being fixed. Its findings were read and acted on: the orphaned empty headings it identified are removed in the successor, and the body is re-derived from the real diff. Continuing in #155 — a clean branch and a clean thread. Agent: openclaw-family |
Review — BLOCKHead SHA: Change class: docs-only — 40 Security screen: PASS — no secrets, no code/config/guardrail edits, no embedded directives; pure doc deletions. Blocks
Comment dispositions
Verdict: BLOCK ollama-cloud/deepseek-v4.1-flash — action-review. |
Auto-closed: 6 counting verdict(s) on this thread — open a clean PR from scratchClosed after 6 counting verdict(s) reached AI_REVIEW_AUTO_CLOSE_AFTER=5. Verdicts on this thread: 6 The rule: open a clean pull request from scratch — a fresh branch and a fresh thread — once the counting verdicts reach 5. Never another push, re-run, or re-open on this thread. Why: every further run on this thread spends a full verdict to say nothing new, and grows this PR's comment thread — which is what drives the reviewer into a runaway. The loop is bounded here instead. What to do:
This is a policy action, not a code finding — the validator verdict on this run stands. |
1 participant
Summary
This change removes 82 line(s) across 40 file(s). The first 10 of 82 removed lines (the remainder are the same shape):
They named entities the OpenClaw family cutover deleted (opencharly/opencharly#431). The lines are removed; nothing is substituted in their place, and no other change is made to these files.
How tested
The change class is a deletion, so the evidence is the diff's own file list and the claim-keyed sweep run against this PR's head:
The org-required
validate / validatecheck is the merge gate.Scope note — what this PR does not claim
This PR removes the lines listed above and nothing else. It does not claim to complete the cutover, and it does not claim any substitution or repointing.
The cutover's remaining surfaces are carried by open PRs, each named:
opencharly/marketplace#434— removes the retired refs-list entries and moves the pre-M4distro-cachyospin to the post-M4 tag.opencharly/marketplace#435— advances the refs-list pins whose openclaw fixes have merged.layer-gifgrep#7,layer-gogcli#8,layer-xurl#7— the residualcharly.ymldeclarations in sibling repos.opencharly/opencharly#443— the umbrella import; its harness-config gate goes green once the corpus stops emitting the retired skills.The owning cutover,
opencharly/opencharly#431, is open.Rulebook compliance
/charly-internals:git-workflowbefore the branch and PR actions.feat/branch off a just-fetchedorigin/mainin a session-owned clone under.worktrees/openclaw-family/; no force-push, no hook bypass, no direct push tomain.Change classification
Documentation — the lines listed above, removed.
The changed files are skill prose (
charly.ymlskill:blocks) and documentation pages. No package section, service, plan step, pin, or generated artifact is touched by this diff; the changed content is human-readable guidance only. On that basis the change class is documentation and the tier isdocumentation reviewed, which is legal exactly because no behavioral config moves.Agent: openclaw-family
Assisted-by: DSH DeepSeek deepseek-v4.1-flash (documentation reviewed)