Skip to content

feat: remove args node type and clean up related resources - #18

Merged
handlename merged 3 commits into
mainfrom
cleanup/remove-args-node
May 22, 2026
Merged

feat: remove args node type and clean up related resources#18
handlename merged 3 commits into
mainfrom
cleanup/remove-args-node

Conversation

@handlename

@handlename handlename commented May 22, 2026

Copy link
Copy Markdown
Owner

Description

Removed the deprecated args node type from the Obsidian Canvas workflow execution plugin "Runestone".

Warning

⚠️ Breaking Change
This release deprecates and completely removes the args node type (runestone.type: args).
Workflows containing args nodes will fail pre-execution validation. Please refer to MIGRATION.md for steps on migrating your workflows to script-based parameter nodes or inline variables.
Note: For backward compatibility, the executor still injects an empty object {} for the args parameter in scripts/templates to avoid immediate runtime ReferenceErrors.

Motivation

With the introduction of the explicit start node marker, the entry points of workflows are now clearly defined. As a result, the args node—which was originally introduced to avoid ambiguity when multiple entry nodes existed—is no longer necessary and has been deprecated.

Key Changes

  • Types, Parser, Validator, & Executor:
    • Removed "args" from RunestoneNodeType.
    • Cleaned up args-specific graph validation rules (such as block requirements and edge constraints) and parsing logic.
    • Removed concurrent execution setup, merging, anyArgsEdgeDismissed checking, and warning logs for the args node from the execution engine.
  • Backward Compatibility:
    • To prevent breaking changes for existing scripts that reference args variables or templating like {{args.key}}, the executor still injects an empty object {} for the args parameter, avoiding runtime errors like ReferenceError.
  • Test Suite Updates:
    • Removed args-specific test cases and updated makeNode helpers in executor.test.ts and validator.test.ts.
  • Documentation Updates:
    • Removed references to the args node from AGENTS.md, REQUIREMENTS.md, GLOSSARY.md, and README.md.
    • Added a migration guide for the args node deprecation in MIGRATION.md (v0.4 section).
  • Skill Definition & Example Vault:
    • Removed args node references in Claude Code skill definition (SKILL.md) and removed args-node.md templates.
    • Cleaned up the example vault (vault.example) by deleting the args-exec-example and refactoring para-note workflow to pass parameters using the upstream config node via the input array rather than the deprecated args node.

Verification Results

  • All 154 unit tests pass successfully.
  • npm run lint completes with no errors or warnings.

@handlename
handlename merged commit f4f45a7 into main May 22, 2026
2 checks passed
@handlename
handlename deleted the cleanup/remove-args-node branch May 22, 2026 02:23
@handlename handlename mentioned this pull request May 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant