Conversation
… README, contrib tooling - NEW Part 25: Architecture Overview (v4.0+) — 15-min primer with mermaid diagram - NEW Part 26: Migration Guide — opinionated v3 → v4 → 4.15-beta.1 upgrade paths + rollback plans - NEW Part 27: Gotchas & FAQ — symptom-indexed troubleshooting - Deprecate Part 16 (custom autoDream) with clear banner pointing to Part 22 - Revamp README: badges, Who-This-Is-For, Start-Here, numbers table, themed navigation, mermaid arch diagram, theme-grouped TOC - Add CONTRIBUTING.md + issue templates (correction/gap/version-bump) + PR template - Add GitHub Action for markdownlint + lychee internal link check - Add .markdownlint.jsonc config tuned for a prose guide - Fix MD001 heading-increment in Part 10 - Fix broken link in Part 16 footer Co-Authored-By: Rob <onerobby@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
- Correct relative path in .github/ISSUE_TEMPLATE/version-bump.md (../../ not ../) - Drop broken file link to scripts/qwen_embed_server_v3.py in Part 10; describe as local install path - Add lychee.toml to exclude templates/ (placeholder user-vault paths) and ephemeral URL patterns Co-Authored-By: Rob <onerobby@gmail.com>
Devin Review flagged: #part-5-orchestration → #part-5-orchestration-stop-doing-everything-yourself (GitHub renders full heading text as the slug; the truncated form silently lands at README top) Co-Authored-By: Rob <onerobby@gmail.com>
|
Devin Review finding addressed in 926-ish: the truncated Side note for the @OnlyTerp: lychee's |
There was a problem hiding this comment.
🟡 Inconsistent UI name: "Control UI" vs "Canvas UI" for the same component
This PR introduces two different names for the same browser UI component. Part 25 (part25-architecture-overview.md:95) authoritatively defines it as "Canvas UI", and Part 26 uses "Canvas UI" consistently (part26-migration-guide.md:115,120,157). However, Part 10's new Copilot section calls it "Control UI", and the README also uses "Control UI" in three new passages (README.md:822, README.md:1194, README.md:1457). All of these references are new in this PR (confirmed: "Control UI" does not exist on master). A reader following Part 25's architecture primer will look for "Canvas UI" and not find it when reading Part 10 or the README checklist sections, and vice versa.
(Refers to line 42)
Was this helpful? React with 👍 or 👎 to provide feedback.
- Fact-check (#5): openclaw tasks -> openclaw flows across Part 24, 26, 27, README; expanded March CVE wave to Feb-Mar with CVE-2026-25253/25157/25158 and CVSS 9.9 WebSocket bypass; added ClawHavoc / Antiy CERT / Trend Micro / Kaspersky framing to Part 23; hedged 'openclaw secrets reload' verb since it varies across 2026.4.x betas and pointed at the Canvas Model Auth status card + models.authStatus gateway method; added OpenClaw docs (clawdocs.org) + changelog (openclawai.io/changelog) links; tightened Part 25 version table (v4.0 Agent OS, v4.1 ClawHub, v4.2 ACP with accurate dates); Unicode em-dash cleanup across files. - Retire Part 16 (#3): deleted part16-autodream-memory-consolidation.md; added one-paragraph retirement note in Part 22 (README) and updated inbound links in README, Part 26, CONTRIBUTING.md; retired-pattern checklist items now point at Part 22 / Part 26. - Decision trees (#4): every part (14 external + 12 README-embedded + the new Part 28) now opens with a 'Read this if / Skip if' callout so readers can self-route. - Glossary (#2): new Part 28 with ~30 terms (MOC, autoDream, Task Brain, ACP, Ralph loop, ClawHub, ClawHavoc, memory-lancedb, LightRAG, DREAMS.md, semantic approval categories, localModelLean, gateway daemon, coordinator protocol, CVE wave, etc.), cross-linked from the themed TOC, 'Navigate By Goal', and 'Primers & references' blocks. Co-Authored-By: Rob <onerobby@gmail.com>
…ns-retire16 * Fact-check + retire Part 16 + decision trees + Part 28 Glossary - Fact-check (#5): openclaw tasks -> openclaw flows across Part 24, 26, 27, README; expanded March CVE wave to Feb-Mar with CVE-2026-25253/25157/25158 and CVSS 9.9 WebSocket bypass; added ClawHavoc / Antiy CERT / Trend Micro / Kaspersky framing to Part 23; hedged 'openclaw secrets reload' verb since it varies across 2026.4.x betas and pointed at the Canvas Model Auth status card + models.authStatus gateway method; added OpenClaw docs (clawdocs.org) + changelog (openclawai.io/changelog) links; tightened Part 25 version table (v4.0 Agent OS, v4.1 ClawHub, v4.2 ACP with accurate dates); Unicode em-dash cleanup across files. - Retire Part 16 (#3): deleted part16-autodream-memory-consolidation.md; added one-paragraph retirement note in Part 22 (README) and updated inbound links in README, Part 26, CONTRIBUTING.md; retired-pattern checklist items now point at Part 22 / Part 26. - Decision trees (#4): every part (14 external + 12 README-embedded + the new Part 28) now opens with a 'Read this if / Skip if' callout so readers can self-route. - Glossary (#2): new Part 28 with ~30 terms (MOC, autoDream, Task Brain, ACP, Ralph loop, ClawHub, ClawHavoc, memory-lancedb, LightRAG, DREAMS.md, semantic approval categories, localModelLean, gateway daemon, coordinator protocol, CVE wave, etc.), cross-linked from the themed TOC, 'Navigate By Goal', and 'Primers & references' blocks. Co-Authored-By: Rob <onerobby@gmail.com> * Refresh for 2026.4.15 stable: Opus 4.7, dreaming storage flip, memory_get excerpt cap, tool-name collision rejection Co-Authored-By: Rob <onerobby@gmail.com> --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Summary
Second pass toward "THE BEST GUIDE ON GITHUB." Builds on #3 with primer/navigation/tooling work: three new flagship parts, a full README hero rebuild, and a CI-backed contribution workflow so the guide stays clean as it grows.
Type of change
What's new
Primers & references (new parts)
README rebuild
Part 16 deprecation
⚠️ DEPRECATEDbanner at the top pointing to Part 22 (memory-core built-in dreaming)part17-coordinator-protocol.mdContributor tooling
CONTRIBUTING.md— what to contribute / what not to / style / how to run checks locallyconfig.ymlroutes OpenClaw bug reports to the upstream repo and steers questions to Part 27 firstpull_request_template.mdwith a review checklist.github/workflows/docs-quality.yml— runs markdownlint-cli2 + lychee (internal link check) on every push / PR.markdownlint.jsonctuned for prose; disables the rules that fight opinionated writing, keeps correctness rulesChecks
markdownlint-cli2 "**/*.md"→ 0 errors./*.mdrefs) → all cleanFlagged items (reviewer sanity-check, same as PR #3)
These are informed guesses based on our discussion, not direct reads of the OpenClaw 2026.4.15-beta.1 schema. Correct me and I'll re-stamp:
memorySearch.provider: "copilot",agents.defaults.experimental.localModelLean, semantic categoriesread-only.*/execution.*/write.*/control-plane.*,openclaw secrets reload,openclaw tasks list/show/audit).npm install -g openclaw@...pin. Replace with the actual install method recommended by OpenClaw docs if different (brew tap, binary, cargo, etc.).https://github.com/openclaw/openclaw/issuesis my best guess for the upstream OpenClaw repo. Update if the real org/repo is different.Sources / context
CONTRIBUTING.mdLink to Devin session: https://app.devin.ai/sessions/df6f8c16f82e448b915735660ed94fb7
Requested by: @OnlyTerp