Skip to content

feat: add standing cable leg curl exercise - #14

Merged
lagrangee merged 1 commit into
mainfrom
codex/standing-cable-leg-curl
Oct 27, 2026
Merged

lagrangee merged 1 commit into
mainfrom
codex/standing-cable-leg-curl

Conversation

@lagrangee

Copy link
Copy Markdown
Owner

Summary

Plans cannot currently prescribe a standing cable leg curl with an ankle strap. Add an active strength Exercise supporting per-side repetitions and external resistance in kg or lb. Existing Exercise IDs and Session snapshots are unchanged; no schema or database migration is needed.

 Exercise Registry
+ standing_cable_leg_curl: per_side, reps, external_load

Evidence

  • Before: The Exercise is absent from the registry and cannot be selected in a Plan Update.
  • After: A synthetic loaded per-side prescription validates; bilateral execution is rejected. Focused tests pass 11/11. npm run check passes type checks and 380 tests with one existing skip on Node 26.5.0.
  • Keep the category-migration test scoped to its 26 historical IDs rather than including later registry additions. Migration SQL is unchanged.

Merge Danger

Door: Two-way before the new Exercise is used; afterward retain its ID for historical resolution.

Blast Radius: Registry

Adds one global movement definition. Contains no personal prescriptions, training records, credentials, or operator receipts. Deploy before using the new ID in a live Plan Update.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-27T11:42:13.114636Z ae4b920 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@lagrangee
lagrangee merged commit 874cb32 into main Oct 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant