Skip to content

docs: correct Sentinel MD5 journal entry scope - #862

Merged
groupthinking merged 1 commit into
mainfrom
fix/sentinel-md5-journal-scope
Jul 17, 2026
Merged

docs: correct Sentinel MD5 journal entry scope#862
groupthinking merged 1 commit into
mainfrom
fix/sentinel-md5-journal-scope

Conversation

@groupthinking

Copy link
Copy Markdown
Owner

What

  • correct the Sentinel MD5 migration journal entry date from 2024-07-09 to 2026-07-09
  • scope the prevention guidance to src/
  • explicitly exclude archived scripts in scripts/archive/

Why

The old PR was stale and unsafe to merge. The underlying MD5 migration is already on main, but the journal entry still overstated repo-wide policy and had the wrong date.

Risk

Low. Documentation-only change.

Closes #651.

Copilot AI review requested due to automatic review settings July 17, 2026 09:47
@vercel

vercel Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
v0-uvai Canceled Canceled Jul 17, 2026 9:47am

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jul 17, 2026
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • [‘architecture-gap’, ‘bug’, ‘ci-cd’, ‘ci/cd’, ‘copilot-rabbit’, ‘documentation’, ‘duplicate’, ‘enhancement’, ‘frontend’, ‘github_actions’, ‘good first issue’, ‘help wanted’, ‘high-priority’, ‘invalid’, ‘javascript’, ‘ml-model’, ‘needs-triage’, ‘pipeline-critical’, ‘placeholder-code’, ‘priority:high’, ‘python’, ‘python:uv’, ‘question’, ‘styling’, ‘tests’, ‘v0’]

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository YAML (base), Repository UI (inherited), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Pro

Run ID: 6682adbe-a12a-46b9-b844-05cd476b0c40

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/sentinel-md5-journal-scope
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch fix/sentinel-md5-journal-scope

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

@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 232defa.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@groupthinking
groupthinking merged commit 60d84de into main Jul 17, 2026
21 of 23 checks passed
@groupthinking
groupthinking deleted the fix/sentinel-md5-journal-scope branch July 17, 2026 09:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Corrects the Sentinel journal entry’s date and narrows its MD5 guidance to active source code.

Changes:

  • Updates the entry date to 2026-07-09.
  • Excludes archived scripts from the src/ MD5 policy.

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

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants