Add VAL alias for input defaults

[GoatInAHat]

Summary

• add a backward-compatible VAL alias that forwards to sg:val-or-default
• extend input helper tests to cover the alias fallback behavior
• update the auto-load smoke check to ensure the helper and alias are present in the bundle

Testing

• ./bundle.sh
• ./tests/auto_load_test.sh
• ./tests/smoke.sh

---

Codex Task

[GoatInAHat]

@codex resolve this

[chatgpt-codex-connector]

Summary

• Updated sg:prompt-real and sg:prompt-int to skip initget when no flags are supplied, preventing invalid-argument errors during interactive prompting. src/sg-load.lspL17-L27

Testing

• ✅ ./tests/auto_load_test.sh
• ✅ ./tests/smoke.sh

View task →

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Parse property lists instead of ignoring non-interactive args

Non-interactive SGEARMAKE calls now accept a property list like (:module 2 :teeth 12 …), but sg:plist-number retrieves values with assoc, which only works on association lists of conses. With the documented plist (e.g., (SGEARMAKE '(:module 1 :teeth 10 ...))), each assoc returns nil so every parameter falls back to the defaults (module 2, teeth 24, etc.), meaning callers cannot override the inputs and the new non-interactive mode silently ignores all provided values.

Useful? React with 👍 / 👎.

[GoatInAHat]

@codex fix this merge conflict

[chatgpt-codex-connector]

Summary

• Guarded sg:prompt-real and sg:prompt-int initget calls to avoid invalid flag usage and added recursive property-list retrieval so non-interactive SGEARMAKE calls honor provided parameters, including center points. src/sg-load.lspL17-L69
• Expanded AutoLISP input helper tests to validate property list parsing behavior alongside existing fallback checks. tests/sg-input-tests.lspL17-L30

Testing

• ✅ ./bundle.sh
• ✅ ./tests/auto_load_test.sh

View task →

[GoatInAHat]

@codex fix this failed check, auto-load smoke test failed. Check the integrity of the check, if the check is functioning as intended, iterate and re-check until it passes. This pr shouldn't have been submitted with this failing.

Run chmod +x tests/auto_load_test.sh
Bundled 4 modules into SmartGears.lsp
Missing expected symbols: (defun VAL

[chatgpt-codex-connector]

Summary

• Added a backward-compatible VAL alias that delegates to sg:val-or-default, ensuring the expected symbol is present in the bundled loader. src/sg-load.lspL4-L15

Testing

• ✅ ./tests/auto_load_test.sh
• ✅ ./tests/smoke.sh

View task →