Skip to content

fix: waive freshness outside git repositories#163

Merged
CryptoJones merged 7 commits into
mainfrom
fix/non-repo-freshness-exemption
Jul 20, 2026
Merged

fix: waive freshness outside git repositories#163
CryptoJones merged 7 commits into
mainfrom
fix/non-repo-freshness-exemption

Conversation

@CryptoJones

Copy link
Copy Markdown
Owner

Summary

  • skip repository freshness requirements when neither cwd nor the action target is in a Git repository
  • clarify the generated Codex bootstrap and guard remediation text
  • add regression coverage and bump omind to 4.2.1

Validation

  • uv run ruff check .
  • uv run mypy src
  • uv run pip-audit
  • uv run pytest -v --cov=omind --cov-report=term-missing (780 passed, 1 skipped; 85.60% coverage)

Proudly Made in Nebraska. Go Big Red! 🌽 https://xkcd.com/2347/

@CryptoJones
CryptoJones merged commit 9949bed into main Jul 20, 2026
@CryptoJones
CryptoJones deleted the fix/non-repo-freshness-exemption branch July 20, 2026 00:20
@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 86b46914-d3c1-4c5d-8a90-7ae78b73322a

📥 Commits

Reviewing files that changed from the base of the PR and between c8dd543 and 8854547.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (6)
  • CHANGELOG.md
  • pyproject.toml
  • src/omind/__init__.py
  • src/omind/guard.py
  • src/omind/seeds.py
  • tests/test_guard.py

Cache: Disabled due to Reviews > Disable Cache setting


📝 Walkthrough

Summary by CodeRabbit

  • Bug Fixes

    • Repository freshness checks are now skipped when neither the working directory nor the action target is inside a Git repository.
    • Updated guidance clarifies when freshness checks and fetch commands are required.
  • Documentation

    • Added release notes for version 4.2.1.
  • Chores

    • Updated the package version to 4.2.1.

Walkthrough

The freshness guidance now waives Git checks when both the working directory and action target are outside a repository. A regression test verifies this behavior, and release metadata is updated to version 4.2.1.

Changes

Non-repository freshness waiver

Layer / File(s) Summary
Freshness guidance and regression coverage
src/omind/guard.py, src/omind/seeds.py, tests/test_guard.py
Guard and bootstrap guidance now describe skipping Git freshness checks outside repositories, and a regression test verifies the action is allowed.
Release metadata and changelog
pyproject.toml, src/omind/__init__.py, CHANGELOG.md
Package version declarations change to 4.2.1, with a changelog entry documenting the freshness waiver.

Estimated code review effort: 1 (Trivial) | ~5 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/non-repo-freshness-exemption

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

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