Skip to content

fix: deliver the fork delta since the 1109 sync wave - #859

Merged
Blogbotana merged 2 commits into
Apibim-com:masterfrom
axisrow:chore/upstream-batch-1109
Nov 10, 2026
Merged

Blogbotana merged 2 commits into
Apibim-com:masterfrom
axisrow:chore/upstream-batch-1109

Conversation

@axisrow

@axisrow axisrow commented Nov 9, 2026 •

Copy link
Copy Markdown
Collaborator

What this changes

Delivers the fork delta accumulated since the #853 delivery as one batch commit on top of
master (the single-batch delivery the fork sync policy prescribes). Two lines, delivered
whole:

The fork-only policy surface (its workflow forms, docs/plan/, visual evidence, fork tooling)
is not part of this delivery; the upstream-owned Python CI tools are untouched.

How you verified it

Never-crash

Every model file, IFC/DSTV import and keystroke is untrusted input (AGENTS.md §4).

  • No new unwrap/expect/panic!/todo! outside tests
  • No indexing or slicing with input-derived positions; allocations sized by input are capped
  • No unsafe

Provenance

Read AGENTS.md §1 and §2 before adding any asset.

  • Nothing taken from Tekla Structures or any Trimble product (assets, code, documentation, tables)
  • No GPL/AGPL code copied in
  • No new assets

Anything a reviewer should look at first

  • The parse loops already reject unknown and repeated keys, so the removed numeric caps only
    ever produced false refusals; the parity (even-length) check is kept.
  • A -h in a value position stays a value, including on the early routes
    (export-view --out -h is not help).

🤖 Generated with Claude Code

The entry point truncated argv at 50 words and the plate/views adapters
capped option pairs below the documented sets (create-plate 28,
create-member 32, update-element 34, define-profile 27, save-view 46,
update-view 39 options), so a fully documented call was refused with a
misleading "options come in name-value pairs". The parse loops already
reject unknown and repeated keys, so the numeric caps only produced false
refusals: drop the caps and the truncation, keep the parity check.

The per-command --help check sat below the early-return routes, so
commands, session, the batch words, export-view, screenshot-view and
export answered --help with a JSON refusal despite the comment promising
otherwise. The check now sits above the route match, where every
registered word passes.

Also document update-element --contour in the registry: the adapter
accepted it, but the inventory did not list it, so the documented count
lagged the whitelist by one; the full-set regression pins the corrected
counts (#552).
…lace

The README install slug referenced the mirror (axisrow/BeamCraft) instead of
the upstream Apibim-com/BeamCraft; point it at the upstream marketplace so
the upstream-facing docs carry the canonical slug.
@Blogbotana
Blogbotana merged commit 27d92a0 into Apibim-com:master Nov 10, 2026
25 checks passed
axisrow added a commit to axisrow/BeamCraft that referenced this pull request Nov 10, 2026
…pibim-com#862 waves and the Apibim-com#859 delivery squash

BeamCraft-Test-Gate: {"version":1,"commit":"d17c66c78a29a7a6eb79597d24c4f21fed0aeba9","tree":"48979ce99d835a7595502b1405fcba222b928e04","base":"d17c66c78a29a7a6eb79597d24c4f21fed0aeba9","merge_parent":"a95d6b7dc77025e90319938b8dad174422ac91f1","os":"macos","arch":"aarch64","scope":"full","ui_independent_reason":null,"ui_tested":true,"rustc":"rustc 1.99.1 (58cf8f9c3 2026-10-13)","cargo":"cargo 1.99.1 (5f94df478 2026-08-27)","nextest":"cargo-nextest 0.9.149 (3fe4ef319 2026-10-20)\nrelease: 0.9.149\ncommit-hash: 3fe4ef31972f24e0c94cfe291718eab0fe678260\ncommit-date: 2026-10-20\nhost: aarch64-apple-darwin","expected_tests":2739,"executed_tests":2739,"ignored_tests":["beamcraft-test-gate::process::process_fixture"],"source_unchanged":true,"stages":[{"name":"format","exit_code":0,"elapsed_ms":1751,"timed_out":false},{"name":"clippy","exit_code":0,"elapsed_ms":8593,"timed_out":false},{"name":"prepare","exit_code":0,"elapsed_ms":25637,"timed_out":false},{"name":"tests","exit_code":0,"elapsed_ms":195244,"timed_out":false},{"name":"doctests","exit_code":0,"elapsed_ms":1487,"timed_out":false},{"name":"docs","exit_code":0,"elapsed_ms":3614,"timed_out":false}]}
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.

2 participants