Skip to content

fix(relay): eslint на v4-модулях — main зламано після мерджу PR#53 до фіксів#55

Merged
vitaliytv merged 1 commit into
mainfrom
claude/v4-fix-followup
Jul 17, 2026
Merged

fix(relay): eslint на v4-модулях — main зламано після мерджу PR#53 до фіксів#55
vitaliytv merged 1 commit into
mainfrom
claude/v4-fix-followup

Conversation

@vitaliytv

Copy link
Copy Markdown
Member

Що сталося

PR #53 (протокол v4 для multi-owner) змерджено о 11:41:51 UTC на коміті 2893506до того, як я запушив фікси чотирьох eslint-порушень (7a5b596, bfab4e8 о 11:43–11:45). Ці два коміти лишилися на вже змердженій гілці й ніколи не потрапили в main; main відтоді червоний на Lint JS.

Що робить цей PR

Той самий фікс, реаплайнутий поверх поточного main:

  • DevPushSink винесено з push.mjs у окремий push-sink.mjs (max-classes-per-file)
  • signing.mjs: Buffer.from(…, 'base64')Uint8Array.fromBase64()
  • Тести: …toString('base64')…toBase64(), [...map.values()]map.values().toArray(), regex-літерали з toThrow/toMatch винесено в module-scope константи

Верифікація

  • bunx oxlint relay / bunx eslint relay — чисто
  • npx vitest run relay67/67
  • doc-files регенеровано (нова push-sink.md, оновлені push.md/signing.md/index.md), text-lint чистий, change-файл relay (patch)

🤖 Generated with Claude Code

…о фіксів)

PR #53 змерджено на коміті 2893506 — до пуша фіксів eslint (module-scope
regex, max-classes-per-file, Uint8Array-API), тому main лишився з чотирма
порушеннями. Той самий фікс, реаплайнутий поверх поточного main:

- DevPushSink винесено з push.mjs у push-sink.mjs (max-classes-per-file)
- signing.mjs: Buffer.from(…, 'base64') → Uint8Array.fromBase64()
- тести: Buffer …toString('base64') → …toBase64(),
  [...map.values()] → map.values().toArray(),
  regex-літерали в toThrow/toMatch — module-scope константи

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@vitaliytv
vitaliytv merged commit e162fb1 into main Jul 17, 2026
4 checks passed
@vitaliytv
vitaliytv deleted the claude/v4-fix-followup branch July 19, 2026 13:45
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