Skip to content

docs(policy): typo fixes in biometric + retention policies - #1448

Merged
POWERFULMOVES merged 1 commit into
mainfrom
docs/policy-typo-fixes
May 11, 2026
Merged

POWERFULMOVES merged 1 commit into
mainfrom
docs/policy-typo-fixes

Conversation

@POWERFULMOVES

Copy link
Copy Markdown
Owner

Summary

Rescues two single-character typo fixes stranded in D:/PMOVES.AI/PMOVES.AI-pr1391 (PR #1391 merged 2026-04-26). Both are markdown/SQL accuracy corrections.

Files

  • docs/biometric-data-handling-policy.md:249 — Shell pipe escape: mount grep (broken) → mount \| grep in the T4 sandbox monitoring table example. Without the escape, markdown renders the table cell with a stray grep floating outside the code fence.
  • docs/data-retention-deletion-policy.md:175 — SQL operator fix: >< in the retention check query. The original query asked for sessions updated_at > 24h ago (the recent sessions, opposite of intent). The corrected query is updated_at < 24h ago, paired with the clarification that the result should be 0 — meaning sessions older than the retention window must already be purged.

Origin

PR #1391 merged 2026-04-26. Both fixes were drafted in the review worktree but never committed.

Test plan

  • Markdown renders the T4 sandbox table cleanly
  • SQL query is logically correct (sessions older than policy window → expect 0)

🤖 Generated with Claude Code

Two stranded edits from D:/PMOVES.AI/PMOVES.AI-pr1391 (PR #1391 merged 2026-04-26).

- docs/biometric-data-handling-policy.md:249
  Shell pipe escape: `mount  grep` -> `mount \| grep` in T4 sandbox
  monitoring table example.

- docs/data-retention-deletion-policy.md:175
  SQL operator fix: `>` -> `<` in retention check query. Sessions older
  than policy window must be purged; expected result is 0.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented May 10, 2026

Copy link
Copy Markdown
Contributor

Warning

Rate limit exceeded

@POWERFULMOVES has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 59 minutes and 59 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c12f2508-0be3-43f6-87c7-92bba0b85345

📥 Commits

Reviewing files that changed from the base of the PR and between 6cf00bb and 62061f3.

📒 Files selected for processing (2)
  • docs/biometric-data-handling-policy.md
  • docs/data-retention-deletion-policy.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/policy-typo-fixes

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 and usage tips.

@POWERFULMOVES
POWERFULMOVES marked this pull request as ready for review May 10, 2026 14:46
POWERFULMOVES added a commit that referenced this pull request May 11, 2026
Consolidates outbound work from a long Z890-CLAUDE session for clean
pickup by 4090-CLAUDE, 5090-CLAUDE, KiloCode, and operator.

Session output (context for downstream lanes):
- 8 PRs in flight: #1432 (+ commit b3bc5f4 Z890 runbook backlink),
  #1440 cross-platform pattern doc, #1441 W0 Substrate brief,
  #1444 issue #1436 staleness fix, #1445 NATS auth fallback,
  #1446 rescue pr1371 (surgical re-do after contamination caught),
  #1447 rescue pr1385 CATALOG accuracy, #1448 rescue pr1391 typos.
- 2 verified-already-done findings (no PR needed): AB-1 A2UI gitlink
  (gitlink at Hardened tip; AB-1 fix 2 commits behind), PBKDF2 (all
  callsites at OWASP-baseline 600,000).
- 1 stash dropped (USB-PROVISIONING-SWEEP, work merged in #1413).

Open lanes for receiving agents:

5090-CLAUDE (mic + voice-activated CHIT):
  - AB-4 keystone: make -C pmoves secrets-funnel-sync (highest-leverage
    blocker per KiloCode P0/P1 handoff; unblocks AB-5/AB-6/§9.4/A2A
    auth verify)
  - Agent Zero v1.13 boot validation (boot-only, no gitlink bump)
  - CHIT trail sign for today's session (PRs + ACKs above)

4090-CLAUDE (cross-fleet operability reach):
  - W0 PR-1..PR-6 from AGNOTE4482PHI.W0-SUBSTRATE.md
  - Worktree review (16 NEEDS_REVIEW characterized in handoff Lane 2,
    grouped: 1 stranded merged-PR (pr1390), 2 codex review trees,
    2 phase deploy branches, 3 k8s/test branches, 5 docs branches,
    3 unrooted feature branches)
  - 8 REMOVE_SAFE worktrees clear for prune when convenient

Operator decisions (cannot be delegated):
  - 3 stashes remaining (CHIT bypass patterns 22 Apr / submodule
    integration docs 5 Apr / supabase bootstrap 1 Apr)
  - Upstream A2UI sync (8 commits behind upstream main on Hardened)
  - 3 W0 brief questions (CIDR convention / Unifi creds / profile naming)
  - KiloCode handoff staleness pattern flagged (2 of 4 quick-wins
    verified already done)

Per Village Rule: receiving agents add CLAIM entries to
AGNOTE4482PHI.t1.md § Active Claim Register before working a lane.

t1.md not edited in this branch to avoid merge conflict with the W0
Substrate CLAIM/RELEASE that lands via PR #1441. The handoff doc
itself is self-contained — receiving agents post their CLAIMs after
#1441 merges.

Trail unsigned locally (CHIT_PASSPHRASE voice-only on 5090; sign on
5090 side per Lane 3 of the handoff).

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@POWERFULMOVES
POWERFULMOVES merged commit 93db2d2 into main May 11, 2026
11 checks passed
@POWERFULMOVES
POWERFULMOVES deleted the docs/policy-typo-fixes branch May 11, 2026 10:44
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