Skip to content

chore(writing-proofread): natural-japanese を v2.0.0、yomiyasu を v3.1.0 に上げる - #275

Merged
paveg merged 2 commits into
mainfrom
chore/japanese-linters-upstream-sync
Nov 13, 2026
Merged

paveg merged 2 commits into
mainfrom
chore/japanese-linters-upstream-sync

Conversation

@paveg

@paveg paveg commented Nov 13, 2026

Copy link
Copy Markdown
Owner

概要

writing-proofread が使う日本語の lint を、上流の最新に上げる。

  • natural-japanese の lint.py を v1.6.4 から v2.0.0 にする。文章の lint で変わったのは、v1.6.5 で足された和語の名詞化の検出(「事実の確かめ」→「確認」)だけ
  • yomiyasu を v2.4.0 から v3.1.0 にする。3.0.0 から、lint と diff は作業フォルダの外のファイルと、点で始まるフォルダの中のファイルを読まずに exit 2 で終わる。prose-reviewer(mktemp の一時ファイルを渡す)、wording-reviewer(.ai/ などを渡す)、回帰テストがそのままでは動かなくなるので、lint は標準入力で、diff は一時フォルダに入って相対名で渡すように直した

v2.5.0 への自動更新 PR #270 は、この PR で不要になったので閉じた。

レビューの観点

  • prose-reviewer.md と wording-reviewer.md の呼び出し方の変更。2 つ目のブロックが別のシェル呼び出しになっても動くように、P を宣言し直し、一時フォルダは 1 つ目のブロックが出したパスをそのまま使う形にした
  • lint.py は上流からそのまま写したもの(linguist-vendored)。中身を読む必要は薄い
  • NOTICE.md の取り込み元の修正は、2 つ目のコミットに入っている

確かめたこと

確認 結果
run-yomiyasu-tests.sh をタグだけ上げて実行(テストを直す前) FAIL: flag.txt: 正本 is flagged as project_term: lint exited 2(作業フォルダの外のパス)
同じテストを呼び出し方を直してから実行 yomiyasu regression check PASSED (9 cases).
run-tests.sh(natural-japanese) Fixture regression check PASSED.。件数は 22 / 29 / 0 / 0 で変わらない
新しい検出 事実の確かめを進めた。 に wago_nominalization が出る
v3.1.0 の標準入力と v2.4.0 のファイル渡しの比較 件数、行番号、ルール(metaphor_verb、slop_vocabulary、project_term)、語の一覧(正本 を拾う、土台 を許す)が同じ
別の評価役(skeptic)に、配備後と同じ .claude/... の配置で、2 つのエージェントの手順を書いてあるとおりに実行させた 反証なし。exit 2 は出ず、比較のためにファイルパスで渡した側だけが exit 2 になった
just fmt-check ✓ All files formatted correctly!

just test は全体では赤い。落ちる 3 つはどれもこの PR の変更の外にある。

  • project-env-trust と excalidraw-diagram(mise のエラー)は、origin/main でも同じように落ちる
  • equity-decision の EDINET のテストは op を 10 秒のタイムアウトで呼ぶので、通ったり落ちたりする。このブランチでも、2 回目は通った

マージのあとに手で確かめること

  • chezmoi apply のあと、prose-reviewer を実際に 1 回動かす

🤖 Generated with Claude Code

paveg and others added 2 commits November 13, 2026 10:52
上流 v1.6.5 で足された和語の名詞化の検出(「事実の確かめ」→「確認」)を取り込む。v2.0.0 のほかの変更は別スキル critical-thinking の追加で、取り込み済みの calques.py、textcore.py、paragraphs.py と fixtures は変わっていない。fixtures の件数も変わらない。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
yomiyasu 3.0.0 から、lint と diff は作業フォルダの外のパスと、点で始まるフォルダの中のパスを読まずに exit 2 で終わる。prose-reviewer は mktemp の一時ファイルを、wording-reviewer は .ai/ などの文書を渡すので、そのまま上げると両方とも動かなくなる。lint は標準入力で渡し、diff は一時フォルダに入って相対名で渡すようにした。回帰テストも同じ呼び出し方にそろえた。

v2.4.0 と比べて、fixtures の件数、語の一覧(正本を拾う、土台を許す)、行番号は変わらない。新しいルール ai_phrase は wording-reviewer の絞り込みにすでに入っている。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Nov 13, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: a2d0d52

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@paveg
paveg merged commit 2c4c39a into main Nov 13, 2026
12 checks passed
@paveg
paveg deleted the chore/japanese-linters-upstream-sync branch November 13, 2026 03:29
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