Skip to content

AI-304: Layout-agnostic shared eslintrc; fix outdated post-create messages#483

Open
xumaple wants to merge 1 commit into
mainfrom
maplexu/AI-304-eslintrc-nested-workflows-and-postcreates
Open

AI-304: Layout-agnostic shared eslintrc; fix outdated post-create messages#483
xumaple wants to merge 1 commit into
mainfrom
maplexu/AI-304-eslintrc-nested-workflows-and-postcreates

Conversation

@xumaple

@xumaple xumaple commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Broaden the shared eslintrc workflow-override glob to src/**/workflows*.ts so the no-node-builtins rule covers nested per-scenario layouts (e.g. openai-agents' src/<feature>/workflows.ts), letting openai-agents use the shared eslintrc and drop out of ESLINTRC_EXCLUDE. All samples' eslintrc copies are re-synced.

Also fixes 8 samples (env-config, dsl-interpreter, eager-workflow-start, standalone-activity, state, nexus-cancellation, nexus-hello, nexus-messaging) whose .post-create told users to run npm scripts that don't exist (start.watch/workflow); each now matches its real scripts and is excluded from the shared post-create.

…d post-create messages

Broaden the shared eslintrc workflow-override glob to src/**/workflows*.ts so the
no-node-builtins rule covers nested per-scenario layouts (e.g. openai-agents'
src/<feature>/workflows.ts). This lets openai-agents use the shared eslintrc and
drop out of ESLINTRC_EXCLUDE; all samples' eslintrc copies are re-synced.

Fix 8 samples whose .post-create told users to run npm scripts that don't exist
(start.watch/workflow): each now has a tailored message matching its real scripts,
and is excluded from the shared post-create.
@xumaple xumaple requested review from a team as code owners June 24, 2026 18:42

@brianstrauch brianstrauch left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting that this was a widespread issue. Thanks for fixing!

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.

3 participants