Skip to content

chore: improve internal tsconfig options for improved dx#1201

Merged
lishaduck merged 67 commits intomainfrom
dotts
Jan 3, 2026
Merged

chore: improve internal tsconfig options for improved dx#1201
lishaduck merged 67 commits intomainfrom
dotts

Conversation

@lishaduck
Copy link
Copy Markdown
Member

@lishaduck lishaduck commented Dec 30, 2025 •
edited
Loading

PR Checklist

Overview

Relands #1174.
Marking as draft, despite being ready for review, to prevent accidental remerges before #1129.

🔥

lishaduck and others added 30 commits December 27, 2025 15:03
@lishaduck
refactor: tsdown
67639f1
@lishaduck
chore: attw
bbabb56
@lishaduck
feat: extract out ts patching package
a024277
@JoshuaKGoldberg
Merge branch 'main'
1c36ef2
@JoshuaKGoldberg
Merge branch 'main'
967a64d
@lishaduck
fix: delete tsconfig.tsbuildinfos again
8d7351f
@lishaduck
improvements
55bcbfb
@lishaduck
fix: knip
a43eb4d
@lishaduck
fix lockfile
73c3a6e
@lishaduck @JoshuaKGoldberg
ci: cleanup
e9d570a 
Co-authored-by: Josh Goldberg ✨ <git@joshuakgoldberg.com>
@lishaduck
fix: typechecking
7918250 
It didn't seem to be running for either @JoshuaKGoldberg or I without the flag 🤔
@lishaduck
chore: drop spelling because I'm here
3270fd0 
Fixes #1161
@lishaduck
fix: wrap test name in quotes 😂
e7e8a06 
Apparently these are typed as string | Function for some reason!?
@lishaduck
ci: create .astro before linting
3516a84
@lishaduck
ci: satiate the knip ✂️
fbead69
@lishaduck
chore: update perfectionist to sort types with values
833060d 
By default, it separates them.
That'll make the next diff bigger, so I turned it off.

Also fixed a pet peeve of mine where it "incorrectly" sorts subpath imports.
@lishaduck
chore: use type specifiers across the board
34b22b0
@lishaduck
chore: import from ts, not js
49e0f98
@lishaduck
chore: enforce ts extensions over js
f5777f9
@lishaduck
feat: erasableSyntaxOnly
f808df1
@lishaduck
Merge branch 'main' into tsdown
e5af340
@lishaduck
fix: let tsdown build references
b9acbd8
@lishaduck
ci: drop now-redundant tsc build
077ff4b 
🤦 I'd thought they'd made build the default, actually they put it behind a flag.
@lishaduck
Merge branch 'tsdown' into dotts
af62d25
@lishaduck
fix: migrate new batch of rules
25cae5d
@lishaduck
chore: js isn't real
4a88c0c
@lishaduck
Merge branch 'tsdown' into dotts
e07721a
@lishaduck
smaller diff
a73a4b6
@lishaduck
fix: revert whitespace changes in tests
7d404d3
@lishaduck
chore: allow ts extensions in site as well
700ad8e
@vercel
Copy link
Copy Markdown

vercel bot commented Dec 30, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
flint Ready Ready Preview, Comment Jan 3, 2026 7:55pm

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 30, 2025 •
edited
Loading

All reports are resolved now. Thanks! ✅

🗺️ This message was posted automatically by OctoGuide: a bot for GitHub repository best practices.

@lishaduck lishaduck mentioned this pull request Dec 30, 2025
Closed
3 tasks
@lishaduck lishaduck changed the title chore: improve internal tsconfig options for improved dx (take 2 📹) chore: improve internal tsconfig options for improved dx Dec 30, 2025
@lishaduck lishaduck mentioned this pull request Dec 30, 2025
Merged
3 tasks
@lishaduck lishaduck changed the title chore: improve internal tsconfig options for improved dx chore: improve internal tsconfig options for improved dx Dec 31, 2025
@lishaduck
chore: disable rewriteRelativeImportExtensions
641d6a1 
TS still seems to rewrite them, but doesn't give the errors.
Go figure.
@lishaduck
fix: use more ts extensions
0c15043 
Forgot about exports 😅
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Jan 3, 2026 •
edited
Loading

⚠️ No Changeset found

Latest commit: 606c66e

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.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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

@lishaduck lishaduck marked this pull request as ready for review January 3, 2026 19:50
@lishaduck lishaduck enabled auto-merge (squash) January 3, 2026 19:50
@lishaduck
Copy link
Copy Markdown
Member Author

lishaduck commented Jan 3, 2026

Self merging once CI passed given everyone approves and this is particularly conflict-prone.

Ditto

@lishaduck
Copy link
Copy Markdown
Member Author

lishaduck commented Jan 3, 2026

I've been trying out Zed because VS Code has been really slow lately, but, boy, I did I botch that merge 😅

@lishaduck lishaduck merged commit f2fa2e8 into main Jan 3, 2026
7 checks passed
@lishaduck lishaduck deleted the dotts branch January 3, 2026 19:58
@lishaduck lishaduck mentioned this pull request Jan 3, 2026
Merged
3 tasks
lishaduck added a commit that referenced this pull request Jan 3, 2026
@lishaduck
fix: files globs (#1249)
01c571a 
<!-- 👋 Hi, thanks for sending a PR to flint! ❤️‍🔥
Please fill out all fields below and make sure each item is true and [x]
checked.
Otherwise we may not be able to review your PR. -->

## PR Checklist

- [x] Addresses an existing open issue: fixes #1248
- [x] That issue was marked as [`status: accepting
prs`](https://github.com/flint-fyi/flint/issues?q=is%3Aopen+is%3Aissue+label%3A%22status%3A+accepting+prs%22)
- [x] Steps in
[CONTRIBUTING.md](https://github.com/flint-fyi/flint/blob/main/.github/CONTRIBUTING.md)
were taken

## Overview

<!-- Description of what is changed and how the code change does that.
-->
Fixes broken merge of #1129 & #1201
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

🛠 Tooling: switch to verbatimModuleSyntax & allowImportingTsExtensions & erasableSyntaxOnly

2 participants

@lishaduck @JoshuaKGoldberg