Skip to content

Remove raw-html.d.ts - #215

Merged
joeldrapper merged 1 commit into
claude/project-thread-uc4zbqfrom
claude/project-thread-no24v7
Oct 13, 2026
Merged

joeldrapper merged 1 commit into
claude/project-thread-uc4zbqfrom
claude/project-thread-no24v7

Conversation

@joeldrapper

Copy link
Copy Markdown
Collaborator

Requested by Joel · project thread

Stacked on #211 (README). #214 (2.0.0) goes on top of this.

Before: src/raw-html.d.ts declared *.html?raw imports as strings for the integration test's fixtures, and it shipped in the npm package because .npmignore publishes src/.

After: the file is gone. vite/client, which tsconfig.test.json already loads, declares *?raw the same way.

How: deleted the file. Both typechecks, oxfmt and oxlint pass.

🤖 Generated with Claude Code

https://claude.ai/code/session_01TuNwthQu98wTBKUo89rtSZ


Generated by Claude Code

@joeldrapper joeldrapper self-assigned this Oct 13, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Codex Review Summary

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

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-13T09:10:58.817774Z 97f8558 New commits
ℹ️ 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.

@joeldrapper joeldrapper mentioned this pull request Oct 13, 2026
@joeldrapper
joeldrapper force-pushed the claude/project-thread-no24v7 branch from ff1a6ae to 97f8558 Compare October 13, 2026 09:09
vite/client, which tsconfig.test.json already loads, declares *?raw imports
as strings, so this declaration was redundant. It also shipped in the npm
package since .npmignore publishes src/.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TuNwthQu98wTBKUo89rtSZ
@joeldrapper
joeldrapper force-pushed the claude/project-thread-no24v7 branch from 97f8558 to 1650f4f Compare October 13, 2026 09:17
@joeldrapper
joeldrapper added this pull request to stack #213 October 13, 2026 09:24
@joeldrapper
joeldrapper merged commit ae7ab17 into main Oct 13, 2026
7 checks passed
@joeldrapper
joeldrapper deleted the claude/project-thread-no24v7 branch October 13, 2026 09:26
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