fix: repair quality gates and land OpenWiki artifacts - #4
Open
svngoku wants to merge 4 commits into
Open
Conversation
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.
Summary
Master's CI is red: the PR #3 merge emptied
docs/exec-plans/active/, which fails the golden-principles gate, andtest_numpy_availablehas been failing because the sandboxed executor crashes on numpy's runtime imports withKeyError: '__import__'. This branch repairs both gates and commits the OpenWiki documentation set that AGENTS.md already advertises.What changed
agentic_internet/tools/code_execution.py): the exec namespace now exposes a guarded__import__. C extensions such as numpy resolve imports through the frame's__builtins__at runtime, so reductions likenp.array([1,2,3]).sum()work again. The guard enforces the AST validator's module blocklist, so aliasing__import__to dodge the AST call check stays blocked.docs/exec-plans/active/.gitkeeprestores the directory the harness structure check requires, sopython3 .opencode/tools/golden_principles.pyand CI pass again.openwiki/wiki (26 files), the scheduled.github/workflows/openwiki-update.ymlworkflow, andCLAUDE.mdland in the repo. The workflow needsBASETEN_API_KEY(plus optional LangSmith keys) as repository secrets.Test plan
All five repo gates pass locally:
ruff check,ruff format --check,mypy,golden_principles, andpytest(198 passed, up from 195 with 1 failing). New regression tests intests/test_code_execution.pycover the aliased-import block and allowed runtime imports.