Skip to content

Conversation

@JonZeolla
Copy link
Member

@JonZeolla JonZeolla commented Sep 2, 2025

Contributor Comments

This auto installs our pre-commit hook config

Blocked on the zenable-mcp move to OAuth, so an api key isn't required at install time, and then we will need to be sure to skip the pre-commit check in CI mode

Testing

git commit -m "Hit me"
Run a Zenable check on all changed files.................................Failed
- hook id: zenable-check
- exit code: 2


================================================
               Welcome to Zenable
        Production-Grade AI Coding Tools
================================================

Detecting files...

==========================================
        CONFORMANCE CHECK COMPLETE
==========================================

Overall Result: FAIL
Checks Run: 1

File: `/Users/jonzeolla/src/zenable/ai-native-python/README.md`
- Check `AIRecommendation-with-rag`: `fail`
  - Finding: The file contains 'FOOF' at the end which appears to be accidental text that should be removed as it serves no purpose and could confuse users.

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

Copy link

@ai-coding-guardrails ai-coding-guardrails bot left a comment

Choose a reason for hiding this comment

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

I've got 1 comment for you to consider

The PR title and description are not accurate. Here are my suggestions:

Title: feat(pre-commit): add Zenable AI hook and update existing hook versions

Description: # Contributor Comments

This PR updates the pre-commit configuration by:

  • Adding a new Zenable AI native Python hook
  • Updating revision hashes for ruff-pre-commit and lychee repositories to their latest versions

These changes are applied to both the root pre-commit config and the cookiecutter template.

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.
List of skipped files due to configuration

Reviewed with 🤟 by Zenable

@JonZeolla JonZeolla enabled auto-merge (squash) September 2, 2025 19:57
Comment on lines -27 to +31
rev: cef607cddb6279091869bcbb8fb4b906fd4c046c # frozen: nightly
rev: ab5026be0f651c65826148206f845cbfabdd840d # frozen: nightly
Copy link
Member Author

Choose a reason for hiding this comment

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

@JonZeolla JonZeolla added the blocked Unable to make progress label Sep 18, 2025
@JonZeolla JonZeolla removed the blocked Unable to make progress label Oct 28, 2025
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.

2 participants