Skip to content

Fix: config key bugs and function module-name handling - #807

Open
tech-vin wants to merge 3 commits into
TheR1D:mainfrom
tech-vin:fix/codebase-audit-improvements
Open

tech-vin wants to merge 3 commits into
TheR1D:mainfrom
tech-vin:fix/codebase-audit-improvements

Conversation

@tech-vin

Copy link
Copy Markdown

The app reads it via cfg.get("SHELL_INTERACTION") and because the dict key has a space, the lookup is returns None, and then raise UsageError("Missing config key: SHELL_INTERACTION") at startup.

Because  the dict key has a space, the lookup  is returns None, and then raise UsageError("Missing config key: SHELL_INTERACTION")  at startup.
@tech-vin tech-vin changed the title Fix SHELL_INTERACTION environment variable retrieval Fix: config key bugs and function module-name handling Oct 28, 2026
1) Bind CACHE_LENGTH  to the CACHE_LENGTH  env var (was reading CHAT_CACHE_LENGTH)
2) Default DEFAULT_TEMPERATURE to "0.0" so the false float no longer triggers "Missing config key" on first run
Use Path(path).stem for module names instead of rstrip(".py"), which mangled names like copy.py and ignored windows path separators

@tech-vin tech-vin left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

  1. Bind CACHE_LENGTH to the CACHE_LENGTH env var (was reading CHAT_CACHE_LENGTH)
  2. Default DEFAULT_TEMPERATURE to "0.0" so the false float no longer triggers "Missing config key" on first run
  3. Fix trailing space in SHELL_INTERACTION key/env var that raised UsageError at startup and ignored the env override.

This branch has not been deployed

No deployments
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