md-analysis-planner — independently scanned and version-tracked by SaferSkills.
SaferSkills independently audited md-analysis-planner (Agent Skill) and scored it 100/100 (green). The audit ran 55 deterministic rules across Security, Supply Chain, Maintenance, Transparency, and Community; it found 0 high-severity and 0 lower-severity findings. The full rule-by-rule trace and per-finding evidence are below. Free, methodology-open.
Findings & checks · 0 flagged
Every scanned point with the score it earned and what moved between them.
First recorded scan — no prior version to compare against.
The primary manifest — the file an agent reads to learn what this artifact does.
Choose the right MD trajectory analyses and prerequisites before writing post-processing code.
| Input | Description | Example |
|---|---|---|
| System | Material or molecular system | oxide glass, liquid water |
| Goals | Analysis goals | rdf,diffusion,coordination |
| Trajectory format | Dump, DCD, XYZ, H5MD, etc. | LAMMPS dump |
| Velocities | Whether velocities are stored | true |
| Stress | Whether stress/virial is stored | true |
| Unwrap needed | Whether atoms cross PBC | true |
| Timestep | fs per saved frame | 10 |
scripts/md_analysis_planner.py emits these fields (JSON under results):
| Field | Description |
|---|---|
analysis_plan | One entry per goal: goal, method, and status |
required_data | Sorted, de-duplicated data needed across all goals |
equilibration_checks | Standard pre-fit equilibration checklist |
pbc_handling | unwrap_needed, minimum_action, format_note |
warnings | Safety-critical caveats and blockers |
status is one of, from most to least severe: blocked > needs time axis > needs review > ready. A more-severe status is never demoted by a less-severe one. For example, a VACF/VDOS goal with no stored velocities reports blocked even when the timestep is also missing (both warnings are still emitted).
Without --json the script prints the plan lines, a Required data: list, a one-line PBC: note, and a Warnings: section (also mirrored to stderr) so the safety-critical caveats are visible even when stdout is piped.
python3 skills/simulation-workflow/md-analysis-planner/scripts/md_analysis_planner.py \
--system "oxide glass" \
--goals rdf,coordination,bond-angle \
--trajectory-format dump \
--unwrap-needed \
--timestep-fs 10 \
--jsonIf velocities, stress, or timestep information is missing, downgrade dependent analyses and report warnings. The script exits with code 2 and a message on stderr for invalid input (empty system, no goals, non-positive or non-finite timestep, or inputs exceeding the size caps below).
This skill plans analysis and prerequisites; it does not parse large trajectories directly.
Do not trust trajectory-derived results until each applicable item below is recorded against the planner's own output:
scripts/md_analysis_planner.py and confirmed no analysis_plan entry is blocked or needs time axis; record any needs review goal and how its custom analysis was resolved.required_data list is actually present in the trajectory (e.g. velocities for VACF/VDOS, stress or virial + strain history for stress-strain) before running the corresponding analysis.pbc_handling.unwrap_needed is true: confirmed positions were unwrapped using cell + image flags before computing displacements (not raw wrapped coordinates).equilibration_checks: discarded the startup transient, confirmed temperature/pressure plateaus, and compared first-half vs second-half property estimates before any transport/thermodynamic fit.| Tempting shortcut | Why it's wrong / what to do |
|---|---|
| "MSD vs time looks straight, so it's diffusive — just fit a line." | A straight-looking MSD can still include ballistic/sub-diffusive transients. Confirm the log-log slope is ~1 over the fit window first, then fit D = lim MSD/(2dt). |
| "D from the simulation is the diffusion coefficient." | PBC self-diffusion is system-size dependent. Apply the Yeh-Hummer 1/L correction with the actual box length L and viscosity eta; report D_0, not D_PBC. |
| "Positions are in the dump, so MSD is fine." | Wrapped coordinates make atoms jump across the box and corrupt displacements. When unwrap_needed is true, unwrap using cell + image flags before any displacement analysis. |
| "No velocities stored, but I can get VACF/VDOS from positions." | The planner marks VACF/VDOS blocked without velocities for a reason. A finite-difference velocity estimate must be explicitly justified (frame spacing, aliasing); otherwise the result is unreliable. |
| "The run finished, so I can fit transport properties on the whole trajectory." | Run completion is not equilibration. Discard the startup transient and verify temperature/pressure plateaus (the equilibration_checks) before fitting. |
| "One window gave a clean number, no need for error bars." | A single window hides correlation-driven variance. Use block averaging or independent trajectories so every transport/thermodynamic value carries an uncertainty. |
Inputs are scalar CLI values and booleans only. plan_md_analysis validates and bounds every field before use; any violation raises ValueError, which main catches to print a message on stderr and exit with code 2:
system must be non-empty (after stripping) and at most 256 characters.trajectory_format at most 256 characters.timestep_fs, if given, must be a positive, finite number (math.isfiniteand > 0).
Goals are not allowlisted: an unrecognized goal is not rejected but is reported with status needs review and an "unknown goal" warning. The --has-velocities, --has-stress, and --unwrap-needed flags are plain booleans and need no validation.
The script reads and writes no files; all I/O is stdin/args -> stdout (plain text or JSON), with warnings additionally mirrored to stderr. It takes no path arguments and opens no trajectory or output files, so there is no filesystem sandboxing concern. There are no per-file size limits because no files are read; input size is instead capped by the field/goal limits above.
Frontmatter allowed-tools is Read, Bash, Write, Grep, Glob:
Bash is used only to run the bundled scripts/md_analysis_planner.py.Read, Write, Grep, and Glob are used only to inspect, edit, and searchthis skill's own files (the script, references, and SKILL.md) when authoring analysis plans; they are not used to touch trajectory data.
eval, exec, os.system, or dynamic imports.it only parses arguments with argparse and computes a plan in-process.
json.dumps (or deterministic plaintext), never interpolated shell.
MAX_GOALS = 64, MAX_SYSTEM_LEN = 256,MAX_FIELD_LEN = 256) bound the work so a planning helper never materializes pathological input.
references/md_analysis_checks.md for analysis prerequisites and failure modes.script_checks to all three eval cases that pin theexact planner output (statuses, sorted required_data, PBC note, and the specific diffusive-regime / Yeh-Hummer / blocked-not-demoted warnings) so the evals discriminate the skill from a from-memory baseline.
surface warnings/required-data/PBC in non-JSON mode, add diffusive-regime and Yeh-Hummer finite-size guidance, and enforce documented input caps.
~30 seconds. Free. No account. Every finding cites a rule and a line of evidence.