Bundle portable skills for Deep Agents - #237
Draft
tdpham2 wants to merge 7 commits into
Draft
Conversation
Check downloaded frontmatter by lines and cover exact byte preservation for LF and CRLF resources. Verified the Windows failure using CRLF copies of the packaged skills; Ruff and the full suite pass (1161 passed, 21 skipped, 2 deselected).
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
Deep Agent skills currently require callers to provision backend files and configure every source, and checkpointed catalogs can hide later updates. This adds bundled
chemgraphandpbs-hpcskills that are available automatically across local, state, store, and custom remote filesystem backends./chemgraph-skills/route usingimportlib.resources.--[no-]deepagent-discover-skillsand corresponding Python/TOML settings. Support both/workspaceand/workspace/composite route keys without mutating the caller's backend.CLI example
chemgraph run --interactive --workflow deep_agent --deepagent-workspace .Ask: “Read the chemgraph and pbs-hpc skills, then prepare a PBS script using dummy values and validate it with bash -n without submitting it.”
The skills guide documents source precedence, source initialization, diagnostics, and filesystem boundaries. Deep Agents uses
read_fileto load skills; file operations route by path, whileexecuteuses the default backend's executor. Templates/helpers must be copied into that executor's filesystem before execution. Remote execution still requires an appropriate backend.Related issues
None.
Type of change
How was this tested?
ruff check .— passed.git diff --check— passed.pytest tests/ -k "not tblite" -q --tb=short— 1,195 passed, 21 skipped, 2 deselected, using Python 3.12 and the project's pinned dependencies.bash -n; local simulations verify GPU assignments for ranks 0–7, literal argument forwarding, MPI launch arguments with spaced paths, and missing-allocation failures.compatibilityfield, which the generic validator does not recognize; the field remains unchanged in the bundled skill.Checklist
mainand targetsmainruff check .passespytest tests/ -k "not tblite"passes