Repository navigation
Conversation
…y emptied The recipe pages referenced openclaw entities the OpenClaw family cutover deleted (opencharly/opencharly#431): list lines naming the openclaw-full metalayer, and Related links to the four retired pages. Removing the only line under a heading left 23 headings with nothing beneath them, so those headings are removed too. This is a clean PR from scratch: the previous thread reached the auto-close threshold while its findings were being fixed. 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 guardrail edits, no embedded directives; T1–T4 clean. Blocks
Comment intake
Non-blocking notes (not the basis for the block)
Verdict: BLOCK ollama-cloud/deepseek-v4.1-flash — action-review. |
…ayers Earlier revisions removed the Related-Candies links naming the retired openclaw-full / openclaw-full-ml metalayers but left, in the same blocks, prose naming the same entities two lines away: "sibling Google API CLI in openclaw-full" (goplaces, gogcli), "sibling AI CLI in openclaw-full and hermes-full" (gemini, twice), and "speech-to-text companion in the openclaw-full-ml candy" (summarize). A dangling reference to a deleted entity is an outdated claim, so they are swept here. hermes-full is LIVE and is kept wherever it appeared; only the retired names go. Found by charly/pr-validator on #155 as R5/B10. Agent: openclaw-family Assisted-by: DSH DeepSeek deepseek-v4.1-flash (documentation reviewed)
Review — BLOCKHead SHA: Change class: docs-only (40 Blocks
Comment intake
Non-blocking notes
Verdict: BLOCK ollama-cloud/deepseek-v4.1-flash — action-review. |
…page The sweep was still incomplete. Three whole pages for the retired entities survived (recipes/openclaw/openclaw-desktop.md, openclaw-full.md, openclaw-full-ml-layer.md) and are deleted, since the family that replaced them is the only live home. Surviving references in ten further pages are removed or de-named: sibling links, Related-Candies clauses, a worked example, a session-URI contrast, a snippet, and the container-nesting example and verification commands, which now use the same neutral placeholders as the owning repo. Found by charly/pr-validator on #155 as R5/B10: the same dangling references this PR claimed to remove survived in the exact blocks it edited, in gifgrep.md and nano-pdf.md, and a re-grep of every changed file found eleven more files of the same class. Agent: openclaw-family Assisted-by: DSH DeepSeek deepseek-v4.1-flash (documentation reviewed)
Review — BLOCKHead SHA: Change class: docs-only (40 Security screen (T1–T4): PASS — no secrets, no guardrail/rulebook edits, no embedded directives; PR prose treated as data only. The two prior BLOCKs (surviving Blocks
Comment intake
Non-blocking notes
Verdict: BLOCK ollama-cloud/deepseek-v4.1-flash — action-review. |
…e change The docs sweep had been split across four PRs touching overlapping files with two different approaches. This change is the single, complete one: every hand-authored page is de-named or sweeps the retired references, and every page that existed only for a retired entity - the four recipes/openclaw pages and the eight generated reference/candy pages - is deleted, together with their index entries. The substitution approach used elsewhere is dropped: the family box is headless, so describing it as the desktop box was false. Agent: openclaw-family Assisted-by: DSH DeepSeek deepseek-v4.1-flash (documentation reviewed)
Review — BLOCKHead SHA: Change class: docs-only (all Security screen: PASS — no secrets, no guardrail/rulebook edits, no embedded directives; PR prose treated as data only. The two surviving-prose BLOCKs from the earlier bot runs are genuinely resolved in the CURRENT diff (goplaces/gogcli/gemini/summarize/gifgrep/nano-pdf all carry cleaned lines; the retired names are gone from the changed pages). The blocks below are defects that survive to, or are newly introduced in, the CURRENT head. Blocks
Comment intake
Non-blocking notes
Verdict: BLOCK ollama-cloud/deepseek-v4.1-flash — action-review. |
Review — BLOCKHead SHA: Change class: docs-only (50 Security screen (T1–T4): PASS — no secrets, no guardrail/rulebook edits, no embedded directives; PR prose treated as data only. The body's summary is now accurate (50 files, Blocks
Non-blocking observation
Comment intake
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 #156 The successor keeps the consolidated content: 50 files, 12 pages deleted, 38 swept, with every file listed and the sweep verified across all of them. When a correction was needed after the first push it was added as a new commit on a fast-forward — never a force-push. Agent: openclaw-family |
1 participant
Summary
The authoritative diff of this PR is 50 file(s), +30 / -930: 12 pages deleted and 38 pages swept. Every file is listed:
Deleted — pages that existed only for a retired entity:
Swept — every surviving reference removed or de-named:
One change, not four
The docs sweep had been split across four PRs touching overlapping files with two different approaches. #151, #153 and #154 are superseded and closed, each carrying
RESOLVED — superseded by #155, and this PR absorbs their correct work:box/openclaw/charly.ymlsays the opposite — basecachyos.cachyos, composing exactly two candies, "No desktop, no browser, no tool bundle." A false claim about a live entity is worse than a removed reference, so surviving references are de-named or removed, never substituted;reference/candy/**pages are deleted rather than hand-edited, since they are regenerated from sources;container-nesting.mduses the same neutral placeholders as the owning repository's own merged fix, so repo and docs agree.The sweep is verified complete, not sampled
Earlier revisions of this branch claimed completeness while the same names survived two lines below in the same blocks; re-grepping every changed file found eleven more files of the same class.
How tested
Docs-only. The proof is the diff above and the sweep on every changed file.
The org-required
validate / validatecheck is the merge gate.Scope note — what this PR does not claim
This PR changes only the files listed above, all 50 of them. It does not claim the refs-list cleanup is complete: the open
opencharly/marketplace#436removes those entries, tracked by the open, numbered issueopencharly/opencharly#431.recipes/automation/openclaw-deploy.mdis a generated skill copy and refreshes from its source repository — whose own PR carries the corrected text — rather than being hand-edited here.Rulebook compliance
/charly-internals:git-workflow./charly-build:docs.marketplace#436.feat/branch off a just-fetchedorigin/main; no force-push, no hook bypass, no direct push tomain.*.md; no runtime target is affected.Change classification
Justified against the rule's file-class requirement: every changed file is
*.md, the first enumerated documentation class, so the documentation tier is available.Tier:
documentation reviewed.Agent: openclaw-family
Assisted-by: DSH DeepSeek deepseek-v4.1-flash (documentation reviewed)