Skip to content

Rename llm-workflows to workflows-llm and drop eslintrc exclusion#481

Open
brianstrauch wants to merge 1 commit into
mainfrom
workflow-streams-eslint-rename
Open

Rename llm-workflows to workflows-llm and drop eslintrc exclusion#481
brianstrauch wants to merge 1 commit into
mainfrom
workflow-streams-eslint-rename

Conversation

@brianstrauch

Copy link
Copy Markdown
Member

Renaming workflow-streams/src/llm-workflows.ts to src/workflows-llm.ts lets it match the shared src/workflows-*.ts eslint glob. As a result, workflow-streams no longer needs a custom .eslintrc.js (now identical to .shared/.eslintrc.js) or an entry in ESLINTRC_EXCLUDE in .scripts/copy-shared-files.mjs.

  • Renamed the file and updated all references (llm-worker.ts, run-llm.ts, test/workflows.test.ts)
  • Reverted .eslintrc.js to match .shared/
  • Removed workflow-streams from ESLINTRC_EXCLUDE

The no-restricted-imports workflow rule is still enforced on the file via the standard glob (npx eslint src/workflows-llm.ts passes).

🤖 Generated with Claude Code

Renaming src/llm-workflows.ts to src/workflows-llm.ts lets it match the
shared workflows-*.ts eslint glob, so workflow-streams no longer needs a
custom .eslintrc.js or an entry in ESLINTRC_EXCLUDE.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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