Skip to content

fix(ci): unblock inherited #4028 lint failures#4039

Merged
AceHack merged 3 commits into
mainfrom
claim/task-mainline-lint-unblock-4028-20260517
May 17, 2026
Merged

fix(ci): unblock inherited #4028 lint failures#4039
AceHack merged 3 commits into
mainfrom
claim/task-mainline-lint-unblock-4028-20260517

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 17, 2026

Unblocks inherited CI failures currently visible on #4028, without widening #4028's tally-fix scope.

What changed:

  • Fix docs/hygiene-history/ticks/2026/05/17/0221Z.md .claude/rules relative links from 5-up to 6-up so they resolve to repo root.
  • Update tools/inventory/amazon-orders-extract.ts to use a typed runtime-only Playwright import loader and strict dedup fallback for noUncheckedIndexedAccess.

Verification:

  • /opt/homebrew/bin/bun tools/hygiene/audit-tick-shard-relative-paths.ts --enforce --baseline tools/hygiene/audit-tick-shard-relative-paths.baseline.json
  • /opt/homebrew/bin/bun install --frozen-lockfile
  • /opt/homebrew/bin/bun --bun tsc --noEmit -p tsconfig.json

Related:

Claim lifecycle:

  • Claim and release commits are both on this branch; net diff is only the two CI-unblock files.

AceHack and others added 3 commits May 16, 2026 22:57
…nblock

Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
…ed CI unblock PR

Co-Authored-By: Codex <noreply@openai.com>
Copilot AI review requested due to automatic review settings May 17, 2026 04:19
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR unblocks inherited CI failures by correcting tick-shard cross-references and making the Amazon orders extractor compatible with strict TypeScript checks.

Changes:

  • Fixes .claude/rules relative links in a hygiene tick shard.
  • Adds a typed runtime-only Playwright loader to avoid static module-resolution failures.
  • Narrows unchecked array access in the extractor’s URL dedup logic.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
tools/inventory/amazon-orders-extract.ts Updates Playwright loading and dedup fallback for strict TypeScript compatibility.
docs/hygiene-history/ticks/2026/05/17/0221Z.md Corrects rule links from the tick shard to repo-root .claude/rules.

@AceHack AceHack merged commit a989de0 into main May 17, 2026
34 checks passed
@AceHack AceHack deleted the claim/task-mainline-lint-unblock-4028-20260517 branch May 17, 2026 04:23
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.

2 participants