Skip to content

[Review] fix: don't import broken agents#1

Open
govambam wants to merge 1 commit intobase-for-pr-1278from
review-pr-1278
Open

[Review] fix: don't import broken agents#1
govambam wants to merge 1 commit intobase-for-pr-1278from
review-pr-1278

Conversation

@govambam
Copy link
Copy Markdown
Owner

@govambam govambam commented Feb 4, 2026

Recreated from writer#1278 for Macroscope review.

Original PR: writer#1278 by @UladzislauK-Writer
Status: closed (merged)

Recreated using squash merge commit - exact merged state preserved.

Original PR: writer#1278

* fix: don't import broken agents

* fix: clear errors
@macroscopeapp
Copy link
Copy Markdown

macroscopeapp bot commented Feb 4, 2026

Add structured error handling to /api/import in writer.serve.get_asgi_app and surface import failures in BuilderHeaderMoreDropdown.vue to prevent importing broken agents

Introduce structured JSON error responses with summary and optional details in /api/import, validate archives before replacement in writer.app_runner.AppRunner, and display server error summaries and traces in the builder import modal in BuilderHeaderMoreDropdown.vue.

📍Where to Start

Start with the /api/import handler writer.serve.get_asgi_app in serve.py, then review import validation in app_runner.py, and the UI error display in BuilderHeaderMoreDropdown.vue.


Macroscope summarized da82279.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants