Summary
The SKIP_FETCH_TOOLS_EXAMPLE environment variable pattern is an anti-pattern. This should be replaced with per-tool configuration.
Context
From PR #148 review comment by @glebedel:
It seems that this should be a per tool configuration rather than based on an env variable
#148 (comment)
Affected Files
examples/human-in-the-loop.ts- Any other files using
SKIP_FETCH_TOOLS_EXAMPLE or similar env-based feature toggles
Current Implementation
if(process.env.SKIP_FETCH_TOOLS_EXAMPLE){// skip behaviour}Proposed Changes
Replace environment variable checks with explicit per-tool or per-instance configuration:
consttoolset=newStackOneToolSet({skipFetch: true,// or similar configuration option});Or use per-tool configuration as appropriate for the use case.
Related
Summary
The
SKIP_FETCH_TOOLS_EXAMPLEenvironment variable pattern is an anti-pattern. This should be replaced with per-tool configuration.Context
From PR #148 review comment by @glebedel:
#148 (comment)
Affected Files
examples/human-in-the-loop.tsSKIP_FETCH_TOOLS_EXAMPLEor similar env-based feature togglesCurrent Implementation
Proposed Changes
Replace environment variable checks with explicit per-tool or per-instance configuration:
Or use per-tool configuration as appropriate for the use case.
Related
feat/migrate-to-pnpm-vitestaddressing env variable anti-patterns