docs: site-config handling memory + phase-progress refresh#18
Conversation
- New reference-site-config-repo memory: how to use the private uhstray-io/ site-config repo — never leak its contents into public repos/memory, additive WisBot-only edits, isolate off main. (Records no actual values from it.) - Refresh deployment progress: file relay 8a/8b merged; Phase 7 + 8c pending infra values (tracked in site-config NEXT-STEPS). Decisions: new secret/services/wisbot, subdomain wisbot.uhstray.io.
There was a problem hiding this comment.
Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.
Once credits are available, reopen this pull request to trigger a review.
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR updates memory and documentation files to track WisBot deployment progress and establish private site-config handling guidelines. It documents Milestone D completion (file relay service), enumerates remaining operational deployment phases, and introduces strict rules for working with the isolated private infrastructure repository. ChangesDeployment Progress and Site-Config Handling
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.claude/memory/reference-site-config-repo.md (1)
1-7: 💤 Low valueConsider reviewing the metadata
typefield.The frontmatter specifies
type: feedback, but this document appears to be a reference guide establishing handling rules for the private site-config repository. Based on naming patterns in MEMORY.md, documents prefixed withreference-might be better categorized astype: referenceortype: projectrather thantype: feedback.Verify whether
feedbackaccurately describes this memory node's purpose in your taxonomy.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.claude/memory/reference-site-config-repo.md around lines 1 - 7, The frontmatter for the memory node named "reference-site-config-repo" incorrectly uses type: feedback; update the metadata `type` to the correct category (e.g., `reference` or `project`) that matches its purpose, and confirm consistency with MEMORY.md naming conventions; specifically change the `type` field in the frontmatter for the node named reference-site-config-repo, run a quick scan for other `reference-` prefixed entries to ensure they use the same `type`, and update documentation or tests that validate node taxonomy if needed.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In @.claude/memory/reference-site-config-repo.md:
- Around line 1-7: The frontmatter for the memory node named
"reference-site-config-repo" incorrectly uses type: feedback; update the
metadata `type` to the correct category (e.g., `reference` or `project`) that
matches its purpose, and confirm consistency with MEMORY.md naming conventions;
specifically change the `type` field in the frontmatter for the node named
reference-site-config-repo, run a quick scan for other `reference-` prefixed
entries to ensure they use the same `type`, and update documentation or tests
that validate node taxonomy if needed.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 676b0aba-a348-4b8a-854b-a345cc1c1970
📒 Files selected for processing (3)
.claude/memory/MEMORY.md.claude/memory/project-deployment-phase-progress.md.claude/memory/reference-site-config-repo.md
📜 Review details
🧰 Additional context used
🪛 LanguageTool
.claude/memory/project-deployment-phase-progress.md
[style] ~25-~25: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ... download); gated on UploadEnabled. - Phase 8b (#17) — hourly retention loop delete...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[grammar] ~32-~32: Use a hyphen to join words.
Context: ...isolated worktree off main + stage specific files (never git add -A). For...
(QB_NEW_EN_HYPHEN)
🔇 Additional comments (9)
.claude/memory/MEMORY.md (1)
6-7: LGTM!.claude/memory/project-deployment-phase-progress.md (4)
21-25: LGTM!
27-28: LGTM!
30-30: LGTM!
32-32: LGTM!.claude/memory/reference-site-config-repo.md (4)
9-9: LGTM!
11-15: LGTM!
17-17: LGTM!
19-19: LGTM!
Memory/docs only.
uhstray-io/site-configrepo: never copy its contents into public repos/commits/PRs/memory (placeholders only), additive WisBot-only edits, work isolated offmain, OpenBao is the runtime source of truth. Contains no actual values from site-config.