Skip to content

fix(workspace_info): guard null workspace results#260

Open
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/workspace-info-null-guard
Open

fix(workspace_info): guard null workspace results#260
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/workspace-info-null-guard

Conversation

@Gujiassh
Copy link

Summary

  • treat [null] workspace responses as a friendly not-found result instead of letting the helper throw a generic error
  • keep the hierarchy helper unchanged and stop before it is called with an empty workspace selection
  • add regression coverage for the nullable workspace response path

Testing

  • npx jest src/core/tools/platform-api-tools/workspace-info-tool/workspace-info-tool.test.ts
  • npx eslint src/core/tools/platform-api-tools/workspace-info-tool/workspace-info-tool.ts src/core/tools/platform-api-tools/workspace-info-tool/workspace-info-tool.test.ts
  • npm run build

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
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.

1 participant