Skip to content

build(lint): align oxlint config with template-typescript - #110

Merged
furedea merged 1 commit into
mainfrom
build/align-lint-config
Nov 10, 2026
Merged

furedea merged 1 commit into
mainfrom
build/align-lint-config

Conversation

@furedea-agent

@furedea-agent furedea-agent Bot commented Nov 10, 2026

Copy link
Copy Markdown
Contributor

概要

linter 設定を template-typescript と揃えます.Refs furedea/agent-config#74

変更

項目 内容 違反数
typescript/consistent-type-imports template-typescript に合わせて error で有効化 0
typescript/no-explicit-any template-typescript に合わせて error で有効化 0
ignorePatterns に .astro/** Astro の生成物.このサイト固有のため理由コメントを付けて保持 0

規則は template と同じ位置(アルファベット順)に置いています.違反は 0 件のためコード修正はありません.

検証

  • pnpm run lint: 0 warnings / 0 errors(--deny-warnings)
  • pnpm run format:check: 通過

規則の対応表

規則 ID 元の場所 判定
typescript/consistent-type-imports template-typescript .oxlintrc.json linter
typescript/no-explicit-any template-typescript .oxlintrc.json, typescript-style の any 禁止 linter

Enable typescript/consistent-type-imports and typescript/no-explicit-any,
and ignore Astro's generated .astro directory. Both rules report 0
violations.

Refs furedea/agent-config#74

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@furedea

furedea commented Nov 10, 2026

Copy link
Copy Markdown
Owner

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-10T09:05:39.972336Z
We couldn’t complete this review. Comment @codex review to try again.
38267ee Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@github-actions

Copy link
Copy Markdown

Review summary: no findings.

This PR only changes .oxlintrc.json. It enables typescript/consistent-type-imports and typescript/no-explicit-any as errors, and adds .astro/** to ignorePatterns. oxlint accepts JSONC comments, so the new comment line is fine. No other tooling in the repo parses this file. The lint script already passes explicit paths that don't include .astro/, so the new ignore pattern is just a safeguard. I couldn't run lint in this environment, so the claim of zero violations comes from the PR description. I found no correctness or security problems, and a config-only change doesn't need new tests.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

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
furedea-com 38267ee Commit Preview URL

Branch Preview URL
Nov 10 2026, 09:05 AM

@furedea
furedea merged commit e22e523 into main Nov 10, 2026
22 of 24 checks passed
@furedea
furedea deleted the build/align-lint-config branch November 10, 2026 10:28
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