Repository navigation
docs(agents): sync shared working rules from ai-rules - #18
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Task authority and verification AGENTS.md, docs/agents/shared-context-and-memory.md |
Updates task authority and reporting rules, browser and device verification, PR-repair procedures, host capability checks, and browser resource limits. |
Memory handling and spending limits AGENTS.md, docs/agents/shared-context-and-memory.md |
Clarifies conditions for continuing memory work, checks before transmitting memory searches or saves, Windows adapter reservation behavior, and bounded task or evaluation batch spending approvals. |
Estimated code review effort: 2 (Simple) | ~10 minutes
Possibly related PRs
- BigSimmo/caring-contacts#17: Updates the automatic memory recall and task-end save policy in
AGENTS.md. - BigSimmo/caring-contacts#14: Defines Supermemory routes, verified namespaces, and credits-based approval rules.
- BigSimmo/caring-contacts#15: Defines recall bounds and credits-based authority for memory operations.
Merge Risk: 
4fdcd
-
No actionable product, runtime, security, or operational risk was found in the policy updates.
Important
Pre-merge checks failed
Please resolve all errors before merging. Addressing warnings is optional.
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Verification Claims |  |
The PR description claims that “Local docs-link and inventory checks pass” but does not name the exact command or check for either result. The description does give a reason for the full docs-maintena… | Update the PR description to list each exact verification command or check and its result, for example Ran <exact docs-link command>: passed and Ran <exact inventory command>: passed. Keep the unavailable check explicit, for example `No… |
-
Check name Status Explanation Title check The title clearly identifies the documentation change and its source. It accurately summarizes the main pull request objective. Description check The description directly explains the synchronization, affected files, preserved project-owned content, main additions, and verification status. It is related to the changeset. Docstring Coverage 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 Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Generated And Sensitive Files PASS — The PR changes only two tracked Markdown files: AGENTS.mdanddocs/agents/shared-context-and-memory.md. The diff contains documentation and policy text only. No secrets, credentials, tokens…Risky Git Or Deployment Actions PASS. The PR adds no force-push, hard reset, destructive clean, branch deletion, shared-branch rebase, or production-data mutation instruction. It permits only ordinary feature-branch publication and … Supabase Project And Schema Safety The pull request changes only AGENTS.mdanddocs/agents/shared-context-and-memory.md. The diff contains no Supabase project reference, schema, SQL, migration, RLS, policy, or database configuratio…Runtime And Package Manager Integrity PASS: The PR changes only AGENTS.md and docs/agents/shared-context-and-memory.md. package.json, package-lock.json, and .nvmrc are unchanged. The existing package.json still requires Node >=24.15.0 <25… Api Route Failure Handling The pull request changes only AGENTS.mdanddocs/agents/shared-context-and-memory.md. The diff contains working-rule and memory-policy documentation changes only. It does not change API routes, se…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 0 files. (2 skipped: 2 unsupported.)
Full details: Generated And Sensitive Files
-
Explanation
PASS — The PR changes only two tracked Markdown files:
AGENTS.mdanddocs/agents/shared-context-and-memory.md. The diff contains documentation and policy text only. No secrets, credentials, tokens, private keys, real.envfiles, logs, caches, build outputs, browser artifacts, database dumps, or machine-local configuration were found. The files are plain text and the PR description clearly justifies the generated documentation sync.
Full details: Risky Git Or Deployment Actions
-
Explanation
PASS. The PR adds no force-push, hard reset, destructive clean, branch deletion, shared-branch rebase, or production-data mutation instruction. It permits only ordinary feature-branch publication and an explicitly authorized push (AGENTS.md:13; docs/agents/shared-context-and-memory.md:17-23). Merge, automerge, migration, sensitive actions, and deployment retain applicable authority, and the text states that a finished agent turn is not deployment evidence (AGENTS.md:13; docs/agents/shared-context-and-memory.md:22-23).
Full details: Supabase Project And Schema Safety
-
Explanation
The pull request changes only
AGENTS.mdanddocs/agents/shared-context-and-memory.md. The diff contains no Supabase project reference, schema, SQL, migration, RLS, policy, or database configuration change. The references to migrations and production authority are general working rules, not database operations.
Full details: Runtime And Package Manager Integrity
-
Explanation
PASS: The PR changes only AGENTS.md and docs/agents/shared-context-and-memory.md. package.json, package-lock.json, and .nvmrc are unchanged. The existing package.json still requires Node >=24.15.0 <25 and npm 11.x, and .nvmrc remains 24. No package-manager switch, lockfile bypass, engine weakening, or force/legacy resolution flag was introduced.
Full details: Api Route Failure Handling
-
Explanation
The pull request changes only
AGENTS.mdanddocs/agents/shared-context-and-memory.md. The diff contains working-rule and memory-policy documentation changes only. It does not change API routes, server actions, RAG, search, ingestion, or provider integrations, so this check is not applicable.
-
Full details: Verification Claims
-
Explanation
The PR description claims that “Local docs-link and inventory checks pass” but does not name the exact command or check for either result. The description does give a reason for the full docs-maintenance check not running, but the passed checks do not meet the required exact-command/check format. The reviewed diff changes only
AGENTS.mdanddocs/agents/shared-context-and-memory.md; the verification claim is in the supplied PR description.Resolution
Update the PR description to list each exact verification command or check and its result, for example
Ran <exact docs-link command>: passedandRan <exact inventory command>: passed. Keep the unavailable check explicit, for exampleNot run: <exact docs-maintenance command>; formatting dependency was missing, and the unchanged baseline fails for the same reason.
Finishing Touches
-
Simplify code
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Updates this repository's copies of Josh's shared working rules to the current reviewed public projection from
BigSimmo/ai-rules(generated byrepository_rule_targets.py, the same logic assync.sh --repos).Changed: only the managed
shared-cloud-working-rulesblock inAGENTS.mdand the shared prefix ofdocs/agents/shared-context-and-memory.md. Any project-owned addendum is preserved byte-for-byte; the generator refuses to overwrite unreviewed edits.Main additions: authorized browser/device testing preference (Chrome, then Edge), reporting missing capability precisely, leaner process guidance, the parallel dev server memory check, and condensed memory-budget wording. No product, clinical or runtime behaviour changes.
Checks: generated from a clean worktree of
origin/main. Local docs-link and inventory checks pass; the full docs-maintenance check could not run locally (a formatting dependency was missing in the temporary worktree; it fails identically without this change), so CI is the authority.Note for local checkouts: the main checkout carries an older uncommitted copy of these two files from an earlier sync. Once this merges, those two local changes are superseded and can be discarded before pulling.
Summary by cubic
Syncs this repo's shared agent working rules from
BigSimmo/ai-rules, touching only the managedshared-cloud-working-rulesblock inAGENTS.mdand the shared prefix ofdocs/agents/shared-context-and-memory.md. The project-owned addendum is preserved byte-for-byte; the generator refuses to overwrite unreviewed edits.Main additions: authorized browser/device testing preference (Chrome, then Edge), precise reporting of missing capability, leaner process guidance, a parallel dev server memory check, and condensed memory-budget wording. No product, clinical, or runtime behavior changes.
Verification and local checkouts: generated from a clean worktree of
origin/main; local docs-link and inventory checks pass, and the full docs-maintenance check could not run locally (missing formatting dependency, fails identically without this change), so CI is the authority. The main checkout still carries an older uncommitted copy of these two files; once this merges, discard those local changes before pulling.Written for commit 4fdcd68. Summary will update on new commits.
Summary by Gitar
This will update automatically on new commits.