Repository navigation
chore(docs): drop references to things the stack removed - #2448
Conversation
- MAPLE_DB_URL: the PGlite data-dir path it configured is gone, nothing read the Env field, and persistence.md and .env.example still described it. - CLAUDE.md and AGENTS.md named SubagentHost, which is not in the source; the review fan-out uses Subagent from @yielded/agent/subagent. - Comments and docs that pointed at nginx.conf, the compose build, the renamedFrom shims, alert-templates and a generated route tree.
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 54 seconds. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
# Conflicts: # docs/infra.md
This comment has been minimized.
This comment has been minimized.
Maple review🟢 Confidence 10/10 · safe to merge. Comment and doc-string cleanup plus one dead Removes the reader-less Before merge
|
|
Merged with 1 step from "Before merge" still open. If it is done, tick it on the review comment; if not, now is the time.
|
Stacked on #2447. Part of the audit stack that starts at #2434.
Small follow-ups the earlier layers left in files they did not own.
MAPLE_DB_URLis gone. The module that turned it into a PGlite data directory was removed in fix: audit first pass (rollup hour bounds, apdex errors, fetch capture, Tinybird 429 retry) #2434, theEnvfield had no reader, anddocs/persistence.mdand.env.examplestill described a persistent local PGlite. Tests and evals use PGlite in memory and need no setting.CLAUDE.mdandAGENTS.mdnamedSubagentHost, which does not exist in the source. The review fan-out usesSubagentfrom@yielded/agent/subagent.nginx.conf, the compose build of the scraper image, therenamedFromshims,alert-templates/, and a generated route tree the formatter config still mentioned..env.examplesaidLlm.tslives inapps/api. It is inapps/ai.Verification
packages/backendtypecheck is clean after removing theEnvfield. Lint is clean on the two code files.Deliberately untouched
app.json,TODO.md, the tracked.context/directory,doctor.config.jsonand the react-doctor skills. These look like leftovers but are yours to decide.lib/semconv, which nothing imports.docs/browser-sdk.md, an older copy of the landing docs page.Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.