Skip to content

Move create_secrets_files env reads into main#2936

Merged
adamtheturtle merged 2 commits intomainfrom
adamtheturtle/env-to-helpers
Feb 17, 2026
Merged

Move create_secrets_files env reads into main#2936
adamtheturtle merged 2 commits intomainfrom
adamtheturtle/env-to-helpers

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Feb 17, 2026

This refactors admin/create_secrets_files.py so helper functions no longer read from os.environ directly. main() now reads the inactive database environment variables once and passes them through as inactive_database_details. _generate_secrets_file_content() was updated to accept that data and use it when writing the inactive key values. No behavior changes are intended beyond centralizing environment access in main().


Note

Low Risk
Low risk refactor limited to a dev/admin script; behavior should be unchanged aside from when missing inactive env vars are detected (now read once in main()).

Overview
Refactors admin/create_secrets_files.py so inactive Vuforia credentials are read once in main() (after load_dotenv) and passed into _generate_secrets_file_content() as inactive_database_details, instead of the helper reading os.environ directly.

The generated secrets files now populate the INACTIVE_* entries from that passed-in dict, improving separation of concerns and centralizing environment access.

Written by Cursor Bugbot for commit d473c78. This will update automatically on new commits. Configure here.

@adamtheturtle adamtheturtle merged commit dfd5e04 into main Feb 17, 2026
9 of 11 checks passed
@adamtheturtle adamtheturtle deleted the adamtheturtle/env-to-helpers branch February 17, 2026 08:26
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

Comments