Repository navigation
docs(agents): sync shared working rules from ai-rules - #3881
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. |
|
Claude encountered an error after 2s —— View job I'll analyze this and get back to you. |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Task authorization and execution guidance AGENTS.md, docs/agents/shared-context-and-memory.md |
The guidance updates browser and device verification, task-local authorization, delegation, task records, and reporting. It also distinguishes project-memory saving requirements from otherwise authorized work. |
Bounded billable-work approvals AGENTS.md, docs/agents/shared-context-and-memory.md |
Approval now covers a named task or evaluation batch with specified services, estimated cost, currency, and maximum spend. The guidance defines when approval must be renewed and requires tracking usage against the cap. |
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Other
Possibly related PRs
- BigSimmo/PsychSift#3845: Changes the same managed rule blocks and covers memory-save and spending approval rules.
- BigSimmo/PsychSift#3781: Changes approval guidance for live database, worker, and OpenAI commands; this PR updates shared authorization guidance.
- BigSimmo/PsychSift#3859: Defines approval requirements for retrieval evaluations; this PR updates the bounded approval policy for evaluation batches.
Suggested reviewers: claude
Merge Risk: 
7f9ab
-
The updated spending guidance remains authoritative. No actionable merge-blocking risk was established.
Pre-merge checks: 
-
Check name Status Explanation Title check The title clearly identifies a documentation sync of shared working rules from ai-rules. It is concise and specific to the main change. Description check The description clearly explains the scope, preserved content, main additions, verification status, and lack of product or clinical behavior changes. It does not use every template heading and does no… 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. Full details: Description check
-
Explanation
The description clearly explains the scope, preserved content, main additions, verification status, and lack of product or clinical behavior changes. It does not use every template heading and does not explicitly state rollback or provider effects, but the missing details are non-critical for this documentation-only change.
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.)
-
Finishing Touches
-
Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
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 the managed shared working rules from
BigSimmo/ai-rulesintoAGENTS.mdanddocs/agents/shared-context-and-memory.md. Only the managed shared blocks change; project-owned content is preserved byte-for-byte. This is documentation-only — no product, clinical, or runtime behavior changes.Key changes
Local checkouts may carry stale uncommitted copies of these two files from an earlier sync; discard them before pulling once this merges. The full docs-maintenance check could not run locally (missing formatting dependency; it fails identically without this change), so CI is authoritative.
Written for commit 7f9ab24. Summary will update on new commits.
Summary by Gitar
This will update automatically on new commits.
Summary by CodeRabbit