Skip to content

Fix args variable in parameter-less custom commands - #1050

Merged
will-lamerton merged 1 commit into
Nano-Collective:mainfrom
ColumbusLabs:fix/parameterless-command-args
Aug 30, 2026
Merged

Fix args variable in parameter-less custom commands#1050
will-lamerton merged 1 commit into
Nano-Collective:mainfrom
ColumbusLabs:fix/parameterless-command-args

Conversation

@ColumbusLabs

Copy link
Copy Markdown
Contributor

Description

Fixes #1035.

Custom commands without declared parameters now receive the aggregate {{args}} variable, matching commands that do declare parameters. The regression test covers a parameter-less command invoked with multiple arguments.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Changeset

  • Added a changeset (pnpm changeset) describing this change for the changelog

Testing

Automated Tests

  • Regression coverage added in executor.spec.ts
  • All existing tests pass (pnpm run test:all — 6,712 passed, 1 skipped)
  • Focused executor tests pass (18 passed)

Manual Testing

  • Tested with Ollama (not applicable)
  • Tested with OpenRouter (not applicable)
  • Tested with OpenAI-compatible API (not applicable)
  • Tested MCP integration (not applicable)

Checklist

  • If this was for an open issue, I was assigned to it (I posted a claim on the unassigned issue before starting)
  • Code follows project style guidelines
  • Self-review completed
  • Documentation updated (not needed; changeset included)
  • No breaking changes
  • Appropriate logging added (not applicable; no new runtime event)

Additional checks: TypeScript typecheck, Biome formatting/lint, build, Knip, and dependency audit all passed. The repository gate skipped Semgrep because it was unavailable locally.

@will-lamerton

Copy link
Copy Markdown
Member

Thanks for this PR @ColumbusLabs - feel free to add yourself as a contributor to our website via a PR which I will approve :)

https://nanocollective.org/contributors
https://github.com/Nano-Collective/organisation

@will-lamerton
will-lamerton merged commit 0e9114b into Nano-Collective:main Aug 30, 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.

[Bug] {{args}} template variable is silently empty in parameter-less custom commands

2 participants