docs: rewrite README to house standard - #162
Conversation
|
Codex review: needs changes before merge. Reviewed August 2, 2026, 8:14 PM ET / August 3, 2026, 00:14 UTC. ClawSweeper reviewWhat this changesThis branch replaces the long README with a concise installation, archive-source, search, sharing, automation, and documentation front door, and adds an Unreleased changelog entry. Merge readinessKeep this PR open for human review because its author is a repository collaborator. The README rewrite is focused and its documented commands, archive modes, and canonical links match the current source and docs; however, the release-owned Priority: P3 Review scores
Verification
How this fits togetherThe README is Discrawl’s first-run documentation surface: it directs users to installation and local/archive ingestion paths, then hands detailed configuration and command guidance to the docs site. The CLI and its local SQLite archive, Git snapshot, and Worker-backed archive modes supply the behavior the README describes. flowchart LR
A[New user] --> B[README front door]
B --> C[Install and quick start]
B --> D[Archive source choice]
D --> E[Discrawl CLI]
E --> F[Local SQLite archive]
E --> G[Git or remote archive]
B --> H[Detailed documentation]
Before merge
Findings
Agent review detailsSecurityNone. Review metrics
Technical reviewBest possible solution: Land the concise README navigation rewrite after removing the Unreleased changelog entry, leaving release notes to the release-owned process. Do we have a high-confidence way to reproduce the issue? Not applicable: this is a documentation-only change. The relevant validation is source and documentation consistency, and the documented CLI/archive paths are present on current main. Is this the best way to solve the issue? No: the README consolidation is a maintainable direction, but updating the release-owned changelog is not the best path for a normal documentation PR and should be removed. Full review comments:
Overall correctness: patch is incorrect AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning high; reviewed against f5074c9bf61f. LabelsLabel changes:
Label justifications:
EvidenceAcceptance criteria:
What I checked:
Likely related people:
Rank-up movesOptional improvements that raise the rating; they are not merge blockers.
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
|
* origin/main: docs: rewrite README to house standard (openclaw#162) # Conflicts: # CHANGELOG.md
Summary
Documentation disposition
No new reference files were needed. The removed command catalog, bot setup, configuration, storage, security, embedding, sharing, and analytics detail already lives under
docs/and at discrawl.sh; the rewritten README links those canonical pages directly.No factual feature material was dropped as obsolete or unverifiable. Repeated reference prose was removed from the README, and claims retained in the shorter version were cross-checked against the Go manifest, CLI help, source, workflows, release assets, Homebrew formula, and existing focused docs.
Verification
make check— passed: tidy/module verification, format, lint, vet, staticcheck, deadcode, gosec, govulncheck, tests, 85.1% coverage, race tests, CLI smoke tests, and six-platform snapshot builds.go build -o bin/discrawl ./cmd/discrawland./bin/discrawl --version— passed.node scripts/build-docs-site.mjs— passed.publish --checkpassed.init/doctor/syncand interactivetui— flags and usage validated against the built binary's real help output; no live Discord credential or interactive terminal was used.openclaw/homebrew-tapand the current GitHub release assets; the host installation was not modified.invalidornot foundoutput.autoreview --mode local— clean, with no accepted/actionable findings.