feat: add Deno as a supported package manager - #339
Merged
Merged
Conversation
Add RFC-001 documenting plan to remove redundant auto-injected CSS, include @tailwindcss/typography by default, and use prose class in MDX templates instead of verbose [&_] wrapper selectors. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…oposal Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Confirmed no hard technical reason for two nested wrapper divs. Added proposed single-div structure, full reference table of all files that need updating, and risk assessment. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Opinionated animation behaviour should not be auto-injected. Users can add view transitions themselves if desired. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Detect Deno via npm_config_user_agent (set by `dx`) and deno.lock lockfile. Maps to `deno install`, `deno task <script>`, and `dx` for the install, run, and dlx commands respectively. Bumps version to 0.1.7-rc.68. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4ef9b80075
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Comment on lines
+52
to
+54
| if (savedUserAgent !== undefined) { | ||
| Deno.env.set("npm_config_user_agent", savedUserAgent); | ||
| } |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
When npm_config_user_agent was originally unset, restoreUserAgent() now deletes it rather than leaving a stale value from tests that set it (e.g. "should detect deno from user agent"). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Use hidden input (asterisks) when prompting for sensitive values like API keys during init. Adds a sync byte-by-byte stdin reader to avoid conflicts with globalThis.prompt, and shows a key icon for sensitive fields. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
3 tasks
10 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
"deno"to thePackageManagertype and detection logicnpm_config_user_agent(set bydx/ Deno 2.6+) anddeno.locklockfiledeno install,deno task <script>, anddxfor install, run, and dlx commandsdx veryfront initwill now seedeno task devinstead ofnpm run devinitprompts, using a sync byte-by-byte stdin reader to avoid conflicts withglobalThis.promptTest plan
🤖 Generated with Claude Code