Skip to content

Tidy public docs: remove local details, move internal notes - #25

Open
ngallodev wants to merge 2 commits into
mainfrom
docs/public-cleanup-20261015
Open

ngallodev wants to merge 2 commits into
mainfrom
docs/public-cleanup-20261015

Conversation

@ngallodev

Copy link
Copy Markdown
Owner

Summary

This repo is linked from the portfolio site. This PR removes local machine details and moves internal working notes to docs/internal/. Documentation only; no code or behavior changes. This is a fork of DeusData/codebase-memory-mcp. Only files the fork added or changed are touched, and the attribution text is unchanged.

Changes

  • Local details: replaced local state and workspace paths and a CI host name in fork-added docs.
  • Deleted: fork-added working artifacts (session handoff, backlog, a dated memory-lab report, and dated CI and reconciliation reports).
  • Moved to docs/internal/: the upstream audit and reconciliation reports, the runtime assurance plan, the migration status note and specs.
  • Links: updated the README link to the migration status note.
  • Not changed: upstream docs, SECURITY.md (its contact is upstream's), AGENTS.md, and the local Jenkins scripts and job XML that use local paths.

Verification

  • No code, test or script references a moved or deleted path.

Replace home-directory state paths and local checkout paths in the
upstream audit, reconciliation report, backlog and handoff with neutral
placeholders, and refer to the CI host instead of its machine name.
Jenkins helper scripts and job XML contain functional local paths and are
left unchanged.
Delete fork-added working artifacts: the session handoff, backlog, memlab
run report, self-hosting UX report, full-suite failure log, CI recovery
plan, consolidated source reconciliation and the first vertical-slice
plan. Move the dated upstream candidate audit and reconciliation report,
the runtime assurance plan, the migration status log and the
upstream-next-three specs into docs/internal/ with a README stating they
are history. Kept in place: README, attribution and security files,
AGENTS.md, configuration and reference docs, the qualification plan,
runbooks and protocols, and docs/agents (a contract test reads them).
Updated the README migration status link and cross-references in the
moved reports. No upstream-authored files were touched.
@coderabbitai

coderabbitai Bot commented Oct 15, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You'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 31 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: ngallodev/codebase-memory-cli/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 25cf1609-85c3-4b03-940c-e812c2f262cf
📥 Commits

Reviewing files that changed from the base of the PR and between 4f6ede9 and 9312ab2.

📒 Files selected for processing: 20
  • BACKLOG.md
  • MEMLAB_ANALYSIS_AND_RUN_REPORT_2026-09-05.md
  • README.md
  • SESSION_HANDOFF.md
  • docs/CLI_FIRST_VERTICAL_SLICE_IMPLEMENTATION_PLAN.md
  • docs/CLI_SELF_HOSTING_UX_REPORT_20260912.md
  • docs/CONSOLIDATED_SOURCE_RECONCILIATION_2026-09-01.md
  • docs/FULL_SUITE_FAILURES_2026-09-04.md
  • docs/GITHUB_CI_RECOVERY_PLAN_2026-09-08.md
  • docs/internal/CLI_MIGRATION_STATUS.md
  • docs/internal/README.md
  • docs/internal/RUNTIME_ASSURANCE_PLAN.md
  • docs/internal/UPSTREAM_308_CANDIDATE_AUDIT_20260912.md
  • docs/internal/UPSTREAM_RECONCILIATION_REPORT_20261002.md
  • docs/internal/specs/upstream-next-three/cache-stores/SPEC.md
  • docs/internal/specs/upstream-next-three/cache-stores/spec.json
  • docs/internal/specs/upstream-next-three/daemon-ownership/SPEC.md
  • docs/internal/specs/upstream-next-three/daemon-ownership/spec.json
  • docs/internal/specs/upstream-next-three/rescript-hang/SPEC.md
  • docs/internal/specs/upstream-next-three/rescript-hang/spec.json
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copy link
Copy Markdown
Owner Author

Review note (2026-11-02): documentation-only PR, but removing root BACKLOG.md and relocating canonical upstream audit/reconciliation reports can break references and ongoing upstream-sync tracking. In particular, current upstream-sync documentation still describes a canonical ledger under docs/, while this PR moves it under docs/internal/. Before merge, perform a repository-wide link/path reference audit (Markdown links, scripts, AGENTS guidance, CI/release docs and portfolio links), reconcile the canonical backlog/ledger location rather than deleting its active authority, and verify no sensitive host/user/path data remains in public docs. No source/runtime behavior changes were identified from the patch listing; that is not sufficient to approve deletion of active project records.

Copy link
Copy Markdown
Owner Author

Re-review: head unchanged (9312ab2), GitHub now reports mergeable=false. This is a merge conflict, not a qualified docs-only merge. Resolve against current main and retain/repoint canonical backlog and upstream-sync ledger references; run broken-link/path audit before requesting merge.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant