Skip to content

Remove archived guidance and project documentation - #377

Draft
BenCodez wants to merge 1 commit into
masterfrom
prep/advancedcore-guidance-removal-20261030
Draft

BenCodez wants to merge 1 commit into
masterfrom
prep/advancedcore-guidance-removal-20261030

Conversation

@BenCodez

@BenCodez BenCodez commented Oct 30, 2026 •

Copy link
Copy Markdown
Owner

Remove the project guidance, agent memory and documentation selected for private archival. Update the retained architecture check to verify the corresponding source structure.

Draft: the private archive update is pending, and actual agent receiving, discovery, compatibility and rollback must be verified before this change is ready. Existing branch-specific guidance and prior history require separate handling.

Validation: 1,722 unit tests and two packaged-artifact checks passed; the retained architecture check and its missing-file control passed.

Summary by CodeRabbit

  • Documentation
    • Removed repository guides covering setup, architecture, storage, rewards, security, packaging, and platform behavior. These reference materials are no longer available in the repository.
    • Removed instructions for AI-assisted review and MEX Inbox and Relay workflows; those repository-specific workflows are no longer documented here.
  • Tests
    • Updated an architecture check to verify the presence of inventory and item files instead of checking for architecture guidance.

@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5efb0a67-c226-4a3b-acec-9cbeaca2a6f7
Commits

Reviewing files that changed from the base of the PR and between 8702337 and f1936fe.

Files selected for processing: 38
  • .agents/skills/mex-inbox/.mex-managed.json
  • .agents/skills/mex-inbox/SKILL.md
  • .agents/skills/mex-inbox/agents/openai.yaml
  • .agents/skills/mex-inbox/references/cli-workflows.md
  • .agents/skills/mex-relay/.mex-managed.json
  • .agents/skills/mex-relay/SKILL.md
  • .agents/skills/mex-relay/agents/openai.yaml
  • .agents/skills/mex-relay/references/cli-workflows.md
  • .github/skills/code-review/tests/architecture-map.sh
  • .mex/.gitignore
  • .mex/AGENTS.md
  • .mex/ROUTER.md
  • .mex/SETUP.md
  • .mex/SYNC.md
  • .mex/config.json
  • .mex/context/architecture.md
  • .mex/context/conventions.md
  • .mex/context/decisions.md
  • .mex/context/rewards.md
  • .mex/context/setup.md
  • .mex/context/stack.md
  • .mex/context/storage.md
  • .mex/events/operations.jsonl
  • .mex/patterns/INDEX.md
  • .mex/patterns/README.md
  • .mex/patterns/reward-queue-change.md
  • .mex/patterns/user-storage-change.md
  • AGENTS.md
  • docs/checked-player-presence.md
  • docs/jar-packaging.md
  • docs/mysql-startup-failure.md
  • docs/reward-gamemode-eligibility.md
  • docs/security-threat-model.md
  • docs/shared-platform-services.md
  • docs/shared-reward-configuration.md
  • docs/shared-runtime-foundation.md
  • docs/shared-sql-user-access.md
  • docs/sql-only-user-storage.md
Files with no reviewable changes: 37
  • .mex/.gitignore
  • .agents/skills/mex-relay/.mex-managed.json
  • .mex/config.json
  • .agents/skills/mex-inbox/.mex-managed.json
  • .mex/patterns/INDEX.md
  • .mex/patterns/README.md
  • .mex/patterns/user-storage-change.md
  • .agents/skills/mex-relay/agents/openai.yaml
  • .agents/skills/mex-inbox/agents/openai.yaml
  • .mex/context/stack.md
  • .mex/SYNC.md
  • .mex/context/setup.md
  • .mex/SETUP.md
  • .mex/events/operations.jsonl
  • .mex/context/conventions.md
  • docs/shared-runtime-foundation.md
  • .mex/AGENTS.md
  • docs/shared-reward-configuration.md
  • .mex/ROUTER.md
  • .mex/context/architecture.md
  • AGENTS.md
  • docs/security-threat-model.md
  • .agents/skills/mex-inbox/references/cli-workflows.md
  • .mex/patterns/reward-queue-change.md
  • docs/mysql-startup-failure.md
  • .agents/skills/mex-inbox/SKILL.md
  • docs/jar-packaging.md
  • docs/sql-only-user-storage.md
  • .mex/context/decisions.md
  • docs/checked-player-presence.md
  • .agents/skills/mex-relay/SKILL.md
  • .mex/context/rewards.md
  • docs/shared-sql-user-access.md
  • .agents/skills/mex-relay/references/cli-workflows.md
  • .mex/context/storage.md
  • docs/shared-platform-services.md
  • docs/reward-gamemode-eligibility.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Recent review details
Context from checks skipped due to timeout: 3
  • GitHub Check: build
  • GitHub Check: Analyze (actions)
  • GitHub Check: Analyze (java-kotlin)
Walkthrough

The PR deletes the MEX Inbox and Relay skills, repository MEX context and pattern files, AGENTS.md, and several technical documents. It also changes architecture-map.sh to check inventory and item source paths instead of architecture statements in AGENTS.md.

Changes: Repository guidance and MEX content

Layer / File(s) Summary
Remove MEX Inbox and Relay skills
.agents/skills/mex-inbox/*, .agents/skills/mex-relay/*
The skill instructions, CLI workflow guides, metadata, and managed manifests are deleted.
Remove MEX repository context
.mex/*, .mex/context/*, .mex/events/*, .mex/patterns/*
The MEX configuration, guidance, context notes, migration records, and pattern pages are deleted.
Remove repository guidance and technical documents
AGENTS.md, .github/skills/code-review/tests/architecture-map.sh, docs/*
The maintainer guide and technical documents are deleted. The architecture-map test now checks inventory directories and item Java files instead of statements in AGENTS.md.

Priority: Low

Estimated code review effort: 2 (Simple) | ~12 minutes

Change: Other

Merge Risk: Minimal · up to f1936

The change removes guidance and documentation while retaining source-structure checks. No concrete merge-blocking defect remains; private archival and compatibility verification are still reported as pending.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: removal of archived guidance and project documentation.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 1 files.

Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • 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.

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