Detach converted imported challenges - #1110
Open
zardus wants to merge 1 commit into
Open
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 31c9fd335d
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
zardus
force-pushed
the
codex/issue-1037-import-conversion
branch
from
July 26, 2026 16:26
7e65fc4 to
a376135
Compare
zardus
force-pushed
the
codex/issue-1037-import-conversion
branch
from
July 26, 2026 18:06
a376135 to
6e9cf91
Compare
zardus
force-pushed
the
codex/ci-harden-dojo-tests
branch
from
July 26, 2026 18:06
cddd17c to
c09419a
Compare
zardus
force-pushed
the
codex/issue-1037-import-conversion
branch
from
July 26, 2026 18:39
6e9cf91 to
7fb6578
Compare
zardus
added a commit
that referenced
this pull request
Jul 26, 2026
## Summary - make the stats-worker cold-start regression deterministic by stopping the worker, clearing the target cache, and verifying its exact cold-start initialization - make database-shell errors fail tests immediately and surface PostgreSQL diagnostics - raise the test-step limit from 15 to 20 minutes; recent remote #1111/#1113 test steps took up to 16m33s ## Scope This PR is strictly CI/test-only. It changes exactly: - `.github/workflows/ci.yml` - `test/test_background_stats.py` - `test/utils.py` It contains no runtime, production, image-build, Compose, or dependency changes. ## Validation - Python syntax check - workflow YAML parse - whitespace check - full single-node and multi-node suites in GitHub Actions ## Stack The seven feature PRs (#1105, #1106, #1108, #1110, #1111, #1112, and #1113) each contain one feature-only commit directly above this branch. Shared schema-startup hardening is isolated in #1115. Image-build patch-runner hardening is isolated in #1116. Neither is a prerequisite for any feature PR. Merge this PR first, then retarget the nine sibling PRs to `master` before deleting this branch.
zardus
force-pushed
the
codex/issue-1037-import-conversion
branch
2 times, most recently
from
July 27, 2026 02:30
614093f to
eeac667
Compare
zardus
changed the title
zardus
force-pushed
the
codex/issue-1037-import-conversion
branch
2 times, most recently
from
July 27, 2026 05:50
ce9c33c to
f9e2ae6
Compare
zardus
force-pushed
the
codex/issue-1037-import-conversion
branch
from
July 27, 2026 17:24
f9e2ae6 to
95b3d49
Compare
Member
Author
|
@ConnorNelson thoughts on the way this guy is checking for imports? It's the only way that doesn't FP on previously-transferred chals without database changes. Probably okay for now? |
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.
Fixes #1037
Summary
importnow creates a destination-owned challenge instead of mutating the source challengeTesting