Add --chinese startup prayer mode - #18
Open
functionstackx wants to merge 1 commit into
Open
functionstackx wants to merge 1 commit into
functionstackx wants to merge 1 commit into
Conversation
functionstackx
requested review from
JordanNanos,
Prathmesh234 and
samharshe
as code owners
September 18, 2026 02:58
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 6a86909. Configure here.
| action="store_true", | ||
| default=argparse.SUPPRESS, | ||
| help="Print the startup prayer in Simplified Chinese (other output is unchanged).", | ||
| ) |
There was a problem hiding this comment.
New flag shadows existing abbreviations
Low Severity
Adding --chinese beside --container and --command makes argparse’s default prefix matching treat --c as ambiguous, so previously valid --c invocations now error instead of selecting the container target or a review command.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit 6a86909. Configure here.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Change
Add
--chineseto print the startup prayer in Simplified Chinese. English remains the default. The flag works before or afteraudit, its profiles, and its targets:Only the prayer heading and verses are translated. Help, confirmation prompts, audit reports, GPU settings, and machine-readable data are not translated or changed by this flag.
Chinese text wraps by terminal display cells rather than character count. Terminals that cannot encode the Chinese text, or are only one column wide, fall back to English. Pipes, inspection commands, and
CLUSTERMAX_PROGRESS=0still suppress the banner.Validation
python -m pytest -qpassed with 1,362 tests and 835 subtests.git diff --checkpassed.Note
Low Risk
Cosmetic startup banner text only; audit logic and structured output are explicitly unchanged, with safe encoding fallback so decoration cannot break a run.
Overview
Adds
--chineseso interactive live audits can show the decorative startup prayer in Simplified Chinese instead of English. The flag is accepted at the top level and on everyauditsubcommand (profiles, targets, review), before or after other arguments;_show_bannerpasses it through toprint_banner(chinese=...).print_bannernow chooses a Chinese title andCHINESE_PRAYERverses when requested, wraps lines with_wrap_prayer_line(display cells viaget_cwidth, English still usestextwrap), and falls back to English if the stream encoding cannot encode the text (including GB18030 when supported) or the terminal is only one column wide. Pipes,CLUSTERMAX_PROGRESS=0, and existing banner-suppression paths behave the same; help, confirmation, reports, and machine-readable output are unchanged.README documents usage and fallback behavior. Tests cover flag placement, rendering, wrapping, encoding fallback, banner suppression, and parity of inspection/YAML output with English.
Reviewed by Cursor Bugbot for commit 6a86909. Bugbot is set up for automated code reviews on this repo. Configure here.