Feature/sync cwapi3d stubs skill - #402
Merged
Merged
Conversation
Derives the binding inventory from CCwAPI3DPythonController.cpp and writes the stub declarations, cadwork types, docs pages and nav entries that are missing from this repo. Machine paths live in the git-ignored config.personal.toml.
The stub sync bumped the pyproject patch blindly, so the published version never tracked the API it was generated against. Read the versionMinor tag in CwAPI3D/include/CwAPI3DVersion.h instead: a higher build number resets the patch (33.322.7 -> 33.328.0), the same one increments it (33.328.0 -> 33.328.1). versionMajor is ignored -- it carries the marketing year while the package major is the product major. A versionMinor below the packaged minor, or a header with no tag at all, falls back to a patch bump and warns rather than emitting a version PyPI has already seen. The missing header is a preflight config error.
Brunner246
force-pushed
the
feature/sync-cwapi3d-stubs-skill
branch
from
August 7, 2026 14:49
4c62a0c to
8331bc7
Compare
Derives the binding inventory from CCwAPI3DPythonController.cpp and writes the stub declarations, cadwork types, docs pages and nav entries that are missing from this repo. Machine paths live in the git-ignored config.personal.toml.
The stub sync bumped the pyproject patch blindly, so the published version never tracked the API it was generated against. Read the versionMinor tag in CwAPI3D/include/CwAPI3DVersion.h instead: a higher build number resets the patch (33.322.7 -> 33.328.0), the same one increments it (33.328.0 -> 33.328.1). versionMajor is ignored -- it carries the marketing year while the package major is the product major. A versionMinor below the packaged minor, or a header with no tag at all, falls back to a patch bump and warns rather than emitting a version PyPI has already seen. The missing header is a preflight config error.
Brunner246
force-pushed
the
feature/sync-cwapi3d-stubs-skill
branch
from
August 10, 2026 07:02
8331bc7 to
9a1112e
Compare
9and3
reviewed
Aug 10, 2026
9and3
left a comment
9and3
left a comment
Contributor
There was a problem hiding this comment.
🔥 Amazing, love the config.toml, I would add only an example and that is all!
Comment on lines
+24
to
+26
| | `pyproject.toml` | `packages` entry for a new module, and `[project].version` | | ||
|
|
||
| ## Invocation |
Contributor
There was a problem hiding this comment.
I would probably just add an example of "translation" from cpp to py here.
…/cwapi3d/cwapi3dpython into feature/sync-cwapi3d-stubs-skill # Conflicts: # .claude/skills/sync-cwapi3d-stubs/scripts/_cpp_bindings.py
…th detailed examples
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
2 participants
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.
No description provided.