-
Notifications
You must be signed in to change notification settings - Fork 0
fix: setup complete_setup should verify company and agent exist #587
Copy link
Copy link
Closed
Labels
prio:mediumShould do, but not blockingShould do, but not blockingscope:smallLess than 1 day of workLess than 1 day of worktype:fixBug fixes and correctionsBug fixes and corrections
Description
Summary
The POST /api/v1/setup/complete endpoint currently only checks that at least one provider is configured before marking setup as complete. It should also verify that a company and at least one agent have been created.
Current behavior
Only checks app_state.provider_registry length > 0.
Expected behavior
Also check:
- Company name is set (via settings or a company service)
- At least one agent exists (via settings "company.agents" key)
Context
Found during PR #584 review. Deferred because it may need new service methods or settings queries.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
prio:mediumShould do, but not blockingShould do, but not blockingscope:smallLess than 1 day of workLess than 1 day of worktype:fixBug fixes and correctionsBug fixes and corrections