Skip to content

fix: Add Claude Code settings and ban .js import extensions#1223

Merged
charlesvien merged 2 commits intomainfrom
03-11-import_and_claude_settings
Mar 13, 2026
Merged

fix: Add Claude Code settings and ban .js import extensions#1223
charlesvien merged 2 commits intomainfrom
03-11-import_and_claude_settings

Conversation

@charlesvien
Copy link
Member

@charlesvien charlesvien commented Mar 13, 2026 •
edited
Loading

  1. Add .claude/settings.json with deny rules for cli and mobile app directories
  2. Untrack .claude/ directory from .gitignore so project settings are shared
  3. Add biome lint rule banning .js extensions in imports (with allowlist for fuse.js, tippy.js, radix-ui)
  4. Remove .js extensions from all existing imports across apps/code, packages/agent, packages/git and packages/shared

@charlesvien Graphite App
Copy link
Member Author

charlesvien commented Mar 13, 2026 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Mar 13, 2026
Merged
Merged
@charlesvien charlesvien force-pushed the 03-11-import_and_claude_settings branch from a85657d to 700855b Compare March 13, 2026 00:46
@charlesvien charlesvien changed the title Scope oauth tokens by org, replace flat project list with org-keyed map fix: Add Claude Code settings and ban .js import extensions Mar 13, 2026
@charlesvien charlesvien marked this pull request as ready for review March 13, 2026 00:47
@charlesvien charlesvien merged commit dc9187f into main Mar 13, 2026
15 checks passed
@charlesvien Graphite App
Copy link
Member Author

charlesvien commented Mar 13, 2026

Merge activity

@charlesvien charlesvien deleted the 03-11-import_and_claude_settings branch March 13, 2026 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jonathanlab jonathanlab jonathanlab approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@charlesvien @jonathanlab