Skip to content

fix: handle pipes without tools - #181

Open
521Peter wants to merge 1 commit into
CommandCodeAI:mainfrom
521Peter:fix/issue-155-optional-tools
Open

521Peter wants to merge 1 commit into
CommandCodeAI:mainfrom
521Peter:fix/issue-155-optional-tools

Conversation

@521Peter

Copy link
Copy Markdown

Summary

  • reuse the normalized hasTools state when running a pipe
  • avoid reading length from an omitted optional tools array
  • add a regression test for running a pipe without tools

Testing

  • pnpm --filter @baseai/core exec vitest --config vitest.node.config.js --run src/pipes/pipes.issue-155.test.ts
  • verified the regression test fails before the fix and passes after it

The package-wide type check currently fails in the pre-existing src/pipes/pipes.test.ts file, and lint cannot resolve the repository's @langbase/eslint-config/library.js configuration.

Fixes #155

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.

tools.length read properties of undefined

1 participant