Skip to content

fix: rename confg -> config in remove_privacy_info and fix f-string#151

Open
ClozyA wants to merge 1 commit intodevfrom
codex/locate-and-fix-spelling-errors-in-python-code
Open

fix: rename confg -> config in remove_privacy_info and fix f-string#151
ClozyA wants to merge 1 commit intodevfrom
codex/locate-and-fix-spelling-errors-in-python-code

Conversation

@ClozyA
Copy link
Copy Markdown
Member

@ClozyA ClozyA commented Apr 9, 2026

Motivation

  • Correct an obvious spelling mistake in the backend config sanitizer to improve readability and prevent future bugs.
  • Ensure the privacy-cleanup code is syntactically correct and robust when constructing %APPDATA% paths on Windows.

Description

  • Renamed the remove_privacy_info parameter and all internal references from confg to config in app/core/config.py.
  • Updated the code that rewrites relative script/config/log paths to use the corrected config variable consistently.
  • Fixed the nested-quote usage in the %APPDATA% f-string by switching inner quotes to single quotes so the expression is syntactically valid.
  • Preserved original behavior of mapping paths to C:/脚本根目录 and formatting %APPDATA% placeholders.

Testing

  • Ran python -m py_compile app/core/config.py, which initially exposed a syntax error from the f-string and then passed after the fix.
  • The final python -m py_compile app/core/config.py check completed successfully.

Codex Task

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant