Repository navigation
chore(deps): pin Harbor to 3c82380 - #243
Merged
Merged
Conversation
Harbor 3c82380 includes 9faf4883 (#3352), which writes Pi's models.json for a custom endpoint with reasoning and the thinking-level map. That replaces the local endpoint handling that was dropped from Harbor-HF. The scrub integration is re-verified at this pin: Trial._emit and Trial._scrub_jobs_dir are unchanged, trial/hooks.py is unchanged, and the new trial writes (stream.json, injected skills) stay in the trial directory or the trial environment. The parent worker image must be republished from this revision before the Space uses it. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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.
This PR pins Harbor to
3c82380859d187957cfd5cd64802b076d9779550, the current upstreammain, up fromfcf27e55. The new revision includes9faf4883(#3352), which makes Harbor's Pi agent writemodels.jsonfor a custom endpoint, withreasoningand the thinking-level map forxhighandmax. That is the upstream replacement for the local Pi endpoint code that was left out of #242.The revision is updated in
pyproject.tomlanduv.lock,launch.py,scrub_before_upload.py,harbor-revision.tsand the tests that assert it.The scrub integration was re-verified at the new pin.
Trial._emitandTrial._scrub_jobs_dirare unchanged, andtrial/hooks.pyis unchanged, so END is still the last trial event. The new trial writes,stream.jsonfrom_write_stream_handleand injected skills from_upload_injected_skills, go into the trial directory or the trial environment only. The docstring inscrub_before_upload.pyrecords this.All checks pass locally:
npm run build, the TypeScript suite (102 files, 1724 tests), the Python agent suite (598 tests), ruff, ty,check:generatedand the public privacy check.Before the control Space runs this revision, the parent worker image has to be republished from it with the "Publish parent worker" workflow, and presets that still pass the removed
endpoint_modeloption to the Pi agent need to move to Harbor's ownmodel_apiandthinkingoptions.🤖 Generated with Claude Code