Skip to content

[codex] Add Effect lint skills and diagnostics#534

Closed
RhysSullivan wants to merge 4 commits intomainfrom
codex/effect-lint-skills
Closed

[codex] Add Effect lint skills and diagnostics#534
RhysSullivan wants to merge 4 commits intomainfrom
codex/effect-lint-skills

Conversation

@RhysSullivan
Copy link
Copy Markdown
Owner

Summary

  • add executor lint diagnostics that point agents at concrete Effect/style skills
  • add and refine skills for typed Effect errors, schema/value-inferred types, promiseExit handlers, package boundaries, and test style
  • codify boundary handling so lint fixes preserve behavior instead of replacing failures with fallback values
  • include representative fixes and suppressions from agent probes to validate the guidance shape

Validation

  • bun run --cwd packages/core/storage-core typecheck
  • bun run --cwd packages/core/storage-core vitest run
  • bunx oxlint -c .oxlintrc.jsonc packages/kernel/runtime-deno-subprocess/src/deno-subprocess-worker.mjs packages/core/cli/src/generators/index.ts packages/core/vite-plugin/src/index.ts scripts/oxlint-plugin-executor/rules/no-unknown-error-message.js --deny-warnings

Notes

  • Draft PR because the broader changed-file lint run still surfaces known backlog in React and MCP plugin paths. Those failures are intentionally left visible for follow-up rule/skill iterations.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 5, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
executor-marketing e9ab55e Commit Preview URL

Branch Preview URL
May 05 2026, 11:05 PM

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 5, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
executor-cloud e9ab55e May 05 2026, 11:06 PM

@RhysSullivan
Copy link
Copy Markdown
Owner Author

Superseded by the split PR stack. Baseline rules/skills are in #535; cleanup changes are being opened as individual stacked draft PRs.

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