Skip to content

chore: use .ts extensions in TypeScript imports - #405

Merged
bartveneman merged 1 commit into
mainfrom
claude/relaxed-volta-8u69nz
Sep 24, 2026
Merged

bartveneman merged 1 commit into
mainfrom
claude/relaxed-volta-8u69nz

Conversation

@bartveneman

Copy link
Copy Markdown
Member

Moves the .js → .ts import extension changes out of #402 into their own PR, so #402 can be rebased on top and only show the theme toggle changes.

Checks: pnpm check (0 errors) and pnpm test:unit (114 passed). lint:ts (mdsvex.config.js) and format:check (src/app.html, group-by-year.test.ts) also fail on main; this PR doesn't cause those failures.

🤖 Generated with Claude Code

https://claude.ai/code/session_01C78rhsv2CRgCow5a5SDq6o


Generated by Claude Code

Replace .js extensions with .ts in imports of TypeScript modules.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C78rhsv2CRgCow5a5SDq6o
@netlify

netlify Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for projectwallace ready!

Name Link
🔨 Latest commit 4f6d2a7
🔍 Latest deploy log https://app.netlify.com/projects/projectwallace/deploys/6ab55a9bd7716b0008ad1e21
😎 Deploy Preview https://deploy-preview-405--projectwallace.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Member Author

Audit is failing, but not because of this PR. pnpm audit reports 10 vulnerabilities (6 high, 4 moderate) in dependencies. One example is js-yaml <4.3.2, pulled in via stylelint > cosmiconfig. This PR doesn't change package.json or pnpm-lock.yaml, and the same check also fails on #402. None of the open PRs fixes it yet, so it needs its own dependency update. I'm not adding one here so this PR stays limited to the import changes.


Generated by Claude Code

@bartveneman
bartveneman merged commit 286c661 into main Sep 24, 2026
10 of 11 checks passed
@bartveneman
bartveneman deleted the claude/relaxed-volta-8u69nz branch September 24, 2026 18:40
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