chore: remove internal governance labels and hardcoded version refs#357
Closed
Wool-xing wants to merge 6 commits into
Closed
chore: remove internal governance labels and hardcoded version refs#357Wool-xing wants to merge 6 commits into
Wool-xing wants to merge 6 commits into
Conversation
added 6 commits
June 12, 2026 04:35
Audit fixes (AUDIT-2026-06-12.md): - P0: C1 config_dir resolution, C4 directory artifact handling, C5 JSON UTF-8, C6 version prefix - P1: H1 MCP dual-path lookup, H2 check_version path, H3 LiteLLM suppress, H4 Prefect cleanup, H5 KB embed degraded flag - P2: M1 sys.path.insert eliminated, M2 loguru bridge, M5 config dual-source, M6 Docker vars, M7 JSON dedup, M8 dynamic paths, M9 locale-aware workspace, M10 LLM env override - P3: L1 CLI loop registration, L2 .env.minimal.example, L3 upload exts, L4 Agent placeholders, L5 Docker tags, L7 /health/deep Architecture: - pyproject.toml includes utils* package - utils/__init__.py: InterceptHandler for loguru bridge - REPL prefix / to ! (Claude Code conflict) - CLI+REPL unified: slash_commands.py single source - Fixed /model /history !insights runtime bugs - deploy/conftest.py: dynamic PROJECT_NAME paths Skins: 22 animal/creature skins with kaomoji + block art + per-skin animation
- ci.yml: remove cd utils + sys.path.insert hack, import as utils.xxx from project root (was causing 'No module named utils' for quality.flaky_detector which uses 'from utils.paths import') - utils/__init__.py: centralized project root path guard, replaces 9 scattered sys.path.insert calls eliminated in audit fix
The removal of 'import sys, os' also removed the 'sys' that sys.exit() depends on. All 10 module imports pass but the script exits 1 due to NameError on sys.
- Remove all 主宪章/铁律/不可逆禁止/决策不可逆 labels across 190+ files - Remove internal §XX section references (ISTQB/hermes external refs kept) - Remove hardcoded version strings (V1.x, V1.0.0, etc.) from comments/docstrings - VERSION file remains single source of truth (V1.0.0) - Preserve all business semantics: '(mock 兜底, selftest 允许)' etc. - All .py imports verified: experts=16, skills=32
Owner
Author
|
Superseded by #359 — minimal fix, all checks pass. |
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
1 participant
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.
Changes
Verification