[codex] Restore CI governance and test baseline#46
Merged
Conversation
helloJamest
marked this pull request as ready for review
July 6, 2026 01:16
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.
PR Type
Background And Problem
The repository's engineering-trust baseline had drifted across AI governance assets, CI workflow coverage, documentation expectations, and local test output. The branch restores those guardrails before moving on to experience or capability work.
Scope Of Change
.env, optional dependencies, Windows file-lock and encoding behavior, and third-party warning noise.newspaper3koptional for search-content fetches and adds a regression test for missing optional dependency behavior.web-gate.AGENTS.mdanddocs/CHANGELOG.md. README was not changed because this is internal governance/test/CI baseline work; detailed changes live in AGENTS, workflow tests, and CHANGELOG.Issue Link
None.
Verification Commands And Results
python -m pytest -m "not network" -q->1445 passed, 17 skipped, 2 deselectedpython scripts\check_ai_assets.py->[ai-assets] OKflake8 . --count --select=E9,F63,F7,F82 --show-source --statistics->0python -c "import pathlib, yaml; [yaml.safe_load(p.read_text(encoding='utf-8')) for p in pathlib.Path('.github/workflows').glob('*.yml')]; print('workflow yaml ok')"->workflow yaml okgit diff --cached --check-> passed before commitnpm.cmd testinapps/finagent-web->43 passed,331 passed | 2 skippednpm.cmd run lintinapps/finagent-web-> passednpm.cmd run buildinapps/finagent-web-> passed, with existing Browserslist staleness and chunk-size warningsCompatibility And Risk
TaskAcceptedalias while avoiding OpenAPI reference collision.newspaper3kremains optional at runtime; missing dependency now degrades article-content fetching to an empty body instead of failing module import.lark_oapi/pkg_resourcesdeprecation noise so project warnings should still surface.scripts/ci_gate.shwas not run as a shell script locally because WSL/bash is unavailable; its key phases were validated through equivalent local checks.Rollback Plan
Revert the branch commits in order, or revert the latest commit
026238fplus the earlier governance commit53be9b2if the whole restoration needs to be backed out.