Skip to content

docs: improve README and fix ai-gatekeeper workflow - #5

Merged
morancj merged 2 commits into
mainfrom
devin/1769692062-readme-improvements
Jan 29, 2026
Merged

morancj merged 2 commits into
mainfrom
devin/1769692062-readme-improvements

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Jan 29, 2026 •

Copy link
Copy Markdown

Summary

This PR makes two changes:

  1. README improvements: Added clearer documentation explaining the repository's purpose and documenting the AI Gatekeeper workflow.

  2. Workflow fix: Added a checkout step to ai-gatekeeper.yaml to resolve the "fatal: not a git repository" error that was causing the gh CLI commands to fail silently. The action is pinned to SHA for security.

Review & Testing Checklist for Human

  • Verify the checkout step doesn't interfere with the existing workflow logic (the gh pr view commands should now work correctly in a git context)
  • Confirm the SHA 34e114876b0b11c390a56381ad16ebd13914f8d5 corresponds to actions/checkout v4
  • Test by creating a Devin-authored PR and verifying the workflow correctly identifies it (previously it was incorrectly skipping with "Not a Devin-generated PR")

Notes

The workflow was previously failing silently because gh commands were running outside a git repository context, causing IS_DEVIN_PR to always be 0 and bypassing the human review enforcement.

Link to Devin run: https://app.devin.ai/sessions/c33c45d83ae743e1a16e2f34738e75bb
Requested by: @morancj

Co-Authored-By: cmoran@dourolabs.xyz <cmoran@dourolabs.xyz>
@devin-ai-integration

devin-ai-integration Bot commented Jan 29, 2026 •

Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@morancj
morancj self-requested a review January 29, 2026 13:10
Co-Authored-By: cmoran@dourolabs.xyz <cmoran@dourolabs.xyz>
@devin-ai-integration devin-ai-integration Bot changed the title docs: improve README with workflow documentation docs: improve README and fix ai-gatekeeper workflow Jan 29, 2026

@morancj morancj 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.

wow such code

@devin-ai-integration

Copy link
Copy Markdown
Author

Thanks! Let me know if you'd like any changes to the PR.

@morancj

morancj commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

Disabled Org ruleset temporarily to merge

@morancj
morancj merged commit eb3c2c1 into main Jan 29, 2026
1 of 2 checks passed
@morancj
morancj deleted the devin/1769692062-readme-improvements branch January 29, 2026 13:20
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