Skip to content
This repository was archived by the owner on Jun 4, 2026. It is now read-only.

Add failed download cleanup prompt and documentation updates#8

Merged
christse merged 1 commit into
mainfrom
feature/sync-cleanup-prompt
Feb 7, 2026
Merged

Add failed download cleanup prompt and documentation updates#8
christse merged 1 commit into
mainfrom
feature/sync-cleanup-prompt

Conversation

@christse
Copy link
Copy Markdown
Contributor

@christse christse commented Feb 6, 2026

Summary

  • Sync command: Fix profile credential loading + add interactive prompt to delete files that fail with 500 errors
  • Milestone command: Remove .boxel-sync.json requirement - now works after pull, push, or sync
  • Documentation: Add pull/push/sync relationship table, workspace organization convention, safety guidelines

Changes

Sync improvements

  • Fix sync command to use profile credentials (was only checking env vars)
  • Track files that fail to download with 500 errors
  • After sync completes, prompt user: "These files may be broken on the server. Delete them from remote? [y/N]"

Milestone improvements

  • Remove .boxel-sync.json requirement
  • Only requires .boxel-history to exist (created by pull/push/sync)

Documentation

  • Add pull/push/sync relationship table explaining command differences
  • Add local workspace organization convention (domain/username/realm)
  • Add safety checkpoint guidance before destructive operations
  • Add 500 error troubleshooting entry

Test plan

  • Run boxel sync on a workspace with broken server files (500 errors)
  • Verify cleanup prompt appears and deletes files when confirmed
  • Run boxel pull then boxel milestone - should work without sync
  • Verify documentation renders correctly

🤖 Generated with Claude Code

Sync improvements:
- Fix sync command to use profile credentials (was only checking env vars)
- Add prompt to delete files that fail with 500 errors during sync
- Track failed downloads and offer cleanup after sync completes

Milestone improvements:
- Remove .boxel-sync.json requirement - now works after pull/push/sync
- Only requires .boxel-history to exist

Documentation:
- Add pull/push/sync relationship table explaining command differences
- Add local workspace organization convention (domain/username/realm)
- Add safety checkpoint guidance before destructive operations
- Add 500 error troubleshooting entry

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@christse christse merged commit 3a07b66 into main Feb 7, 2026
@christse christse deleted the feature/sync-cleanup-prompt branch February 13, 2026 17:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant