the_og/better-claude-config
1
0
Fork 0
Code Issues 8 Pull Requests 3 Actions Packages Projects Releases 4 Wiki Activity

fix: import every dropped .json (aggregate counts) + 5 MB size guard (#39) #45

Merged
the_og merged 1 commits from fix/39-drop-import-all into main 2026-07-12 13:00:44 -04:00
Conversation 0 Commits 1 Files Changed 1
+25 -6
the_og commented 2026-07-12 12:59:46 -04:00
Owner
Copy Link
Copy Source

Closes #39.

  • dropEvent no longer breaks after the first file: every dropped .json is imported, with a single undo snapshot and one aggregate status line ("Imported N added, M replaced from K file(s).").
  • Files over 5 MB (MAX_DROP_IMPORT_BYTES) are skipped with a warning naming the file — a full ~/.claude.json with history is parsed on the UI thread otherwise.
  • Per-file parse failures keep the existing warning-and-continue behavior.

Tests + ruff clean. (Implemented by a delegated agent; reviewed.)

🤖 Generated with Claude Code

Closes #39. - `dropEvent` no longer `break`s after the first file: every dropped `.json` is imported, with a single undo snapshot and one aggregate status line ("Imported N added, M replaced from K file(s)."). - Files over 5 MB (`MAX_DROP_IMPORT_BYTES`) are skipped with a warning naming the file — a full `~/.claude.json` with history is parsed on the UI thread otherwise. - Per-file parse failures keep the existing warning-and-continue behavior. Tests + ruff clean. (Implemented by a delegated agent; reviewed.) 🤖 Generated with [Claude Code](https://claude.com/claude-code)
the_og added 1 commit 2026-07-12 12:59:47 -04:00
fix: import every dropped .json (aggregate counts) + 5 MB size guard (#39)
CI / Lint (ruff) (pull_request) Successful in 7s
Details
CI / Tests (py3.10) (pull_request) Successful in 8s
Details
CI / Tests (py3.12) (pull_request) Successful in 9s
Details
d95db2b026 
Closes #39

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
the_og merged commit b485357cd5 into main 2026-07-12 13:00:44 -04:00
the_og deleted branch fix/39-drop-import-all 2026-07-12 13:00:44 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
P0
P1
P2
blocked
Waiting on an external thing before it can move.
 decision
A decision record (D-007 style) — settled call, referenceable by number from any repo.
 needs-decision
Waiting on AJ's call.
 next
Current focus / next actionable item.
 status
The single pinned STATUS issue for this repo — live state, next action, open items, decisions needed.
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
the_og
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: the_og/better-claude-config#45
Delete Branch "fix/39-drop-import-all"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?