Skip to content

Conversation

@konard
Copy link
Member

@konard konard commented Sep 9, 2025

Summary

  • Added exitCode as an alias for the code property in all result objects
  • Provides better compatibility and convenience for users who expect exitCode
  • Maintains full backward compatibility with existing code property

Implementation Details

  • Updated createResult function to include exitCode property
  • Modified all direct object creations with code property to include exitCode
  • Updated virtual command execution path to include exitCode property
  • Added comprehensive tests for exitCode alias functionality

Test plan

  • Added dedicated test suite (tests/exitcode-alias.test.mjs) covering:
    • Basic exitCode alias functionality
    • Matching values between code and exitCode
    • Success and failure scenarios
    • Pipeline operations
  • Verified all existing tests still pass (no regressions)
  • Tested both virtual command path and real shell command path

🤖 Generated with Claude Code


Resolves #36

Adding CLAUDE.md with task information for AI processing.
This file will be removed when the task is complete.

Issue: #36
@konard konard self-assigned this Sep 9, 2025
konard and others added 2 commits September 9, 2025 21:07
- Added exitCode as an alias for the code property in all result objects
- Updated createResult function to include exitCode property
- Modified all direct object creations with code property to include exitCode
- Updated virtual command execution path to include exitCode
- Added comprehensive tests for exitCode alias functionality
- Ensures backward compatibility while providing the requested alias

Fixes #36

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@konard konard changed the title [WIP] Add exitCode alias for code Add exitCode alias for code property Sep 9, 2025
@konard konard marked this pull request as ready for review September 9, 2025 18:19
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.

Add exitCode alias for code

2 participants