style: enforce the 100-column baseline repo-wide (E501) - #249
Merged
Merged
Conversation
Whitespace/paren reflow and implicit string-literal concatenation only; SQL triple-quotes converted to adjacent literals with whitespace-only diffs; prompt/template strings byte-verified identical
Same mechanical discipline: whitespace/paren reflow, implicit concatenation for long CJK literals; instruction strings in the awakening triggers and defectify prompt byte-verified identical
Assertion/call reflow and implicit string concatenation; identity placeholder template byte-verified identical
Whitespace/paren reflow plus one hoisted payload local; covered by tests/unit/scripts/test_deploy_v4.py
style.md already declares the line-length 100 baseline; pyproject was the lagging side. Repo-wide zero E501 verified by the preceding fold commits
…iptions Independent CR caught four implicit-concat folds leaving a trailing comma, turning description strings into 1-element tuples and emitting JSON arrays to providers when the opt-in tool is enabled. File is now AST-identical to origin/dev. Adds a schema-type guard test over all registered tools
Owner
Author
Bot Review 处置记录(降智处置预案)PR 开出时(02:36Z)Bot 的 review job 已入队,但该 job 于 02:59Z 以
留痕:失败 job |
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.
概要
执行长文件审计 §六.4 的仓库级决策(方案 A):移除
pyproject.toml的ignore = ["E501"],并把全仓存量超长行折叠到 style.md 早已声明的 100 列基线。纯机械变更,无任何语义/文案/行为改动。变更内容
style(llm)→style(src,scripts)→style(tests)→style(prod.example)→ 末位删除 lint ignore。等价性验证
scheduled_messages.py、scheduled_cron.pyfixture、daily_summary_plugin.py:382(仅空白/换行重排);service_parts/health.py(f-string 插值内 94 字符调用,3.11 无法折行,先赋局部变量再拼接,运行时输出逐字节相同);prod.example/deploy-state.py(payload 提升局部变量)。.venv/bin/ruff check .(含 E501)全绿;全量 pytest 2090 全绿;TOML 样例校验通过。独立 CR 处置(ee7acad)