Skip to content

feat(mcp): pull API keys from environment at setup time#36

Merged
JonZeolla merged 3 commits into
mainfrom
use-zenable-api-key-if-set
Jul 10, 2025
Merged

feat(mcp): pull API keys from environment at setup time#36
JonZeolla merged 3 commits into
mainfrom
use-zenable-api-key-if-set

Conversation

@JonZeolla

Copy link
Copy Markdown
Member

Contributor Comments

Now if you set certain env vars at repo generation time (currently just ZENABLE_API_KEY but we plan to support more), it will auto-populate your .envrc file.

This is purposefully undocumented for now, as we test it. We will add docs when this feature goes GA

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.=

@JonZeolla JonZeolla enabled auto-merge (squash) July 10, 2025 12:11

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I've got 2 comments for you to consider

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

Title: feat(mcp): pull API keys from environment at setup time + update pre-commit config

Description: # Contributor Comments

Main Feature

Now if you set certain env vars at repo generation time (currently just ZENABLE_API_KEY but we plan to support more), it will auto-populate your .envrc file.

This is purposefully undocumented for now, as we test it. We will add docs when this feature goes GA

Additional Changes

  • Updated pre-commit configuration by removing verbose flag from ruff linter and updating revision hashes

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.

Reviewed with 🤟 by Zenable

Comment thread hooks/post_gen_project.py
@JonZeolla JonZeolla merged commit 779544d into main Jul 10, 2025
7 checks passed
@JonZeolla JonZeolla deleted the use-zenable-api-key-if-set branch July 10, 2025 12:45
@JonZeolla JonZeolla restored the use-zenable-api-key-if-set branch July 10, 2025 14:33
@JonZeolla JonZeolla deleted the use-zenable-api-key-if-set branch July 10, 2025 14:34
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