Skip to content

fix(extensions-library): bulk manifest corrections across 19 services#554

Open
yasinBursali wants to merge 1 commit intoLight-Heart-Labs:resources/devfrom
yasinBursali:ext/fix-manifest-bulk-corrections
Open

fix(extensions-library): bulk manifest corrections across 19 services#554
yasinBursali wants to merge 1 commit intoLight-Heart-Labs:resources/devfrom
yasinBursali:ext/fix-manifest-bulk-corrections

Conversation

@yasinBursali
Copy link
Contributor

What

Bulk corrections to 19 service manifests fixing three categories of issues:
invalid gpu_backends values, malformed feature declarations, and missing
compose_file fields.

Why

Several manifests contain schema-invalid data that breaks validation:

  • gpu_backends: [none] and gpu_backends: [] are not valid enum values
    (schema allows: amd, nvidia, apple, all)
  • Features declared as dicts nested under service: instead of top-level
    arrays are invisible to consumers
  • Missing compose_file prevents the compose resolver from locating service
    compose files

How

A. Remove invalid gpu_backends (8 service-level + 5 feature-level)
Services: aider, baserow, chromadb, crewai, gitea, label-studio, milvus,
open-interpreter, paperless-ngx, piper-audio, sillytavern, weaviate

B. Convert nested dict features → top-level array (7 services)
Services: bark, baserow, crewai, forge, frigate, paperless-ngx, weaviate
Each converted feature includes all schema-required fields (id, name,
description, icon, category, requirements, priority). GPU services include
gpu_backends on every feature matching the service-level declaration.

C. Add missing compose_file (4 services)
Services: anythingllm, flowise, langflow, librechat

Scope

All changes are within resources/dev/extensions-library/services/.

Testing

  • All 33 manifests parse as valid YAML
  • All converted features pass schema required-fields check against
    service-manifest.v1.json
  • No remaining invalid gpu_backends values in any manifest
  • No remaining nested dict features under service:
  • No remaining services with compose.yaml missing compose_file
  • No secrets in diff

Merging Order

This PR can be merged independently. If any of the following PRs merge first,
a minor rebase may be needed (they touch different sections of overlapping files):

🤖 Generated with Claude Code

Lightheartdevs
Lightheartdevs previously approved these changes Mar 22, 2026
Copy link
Collaborator

@Lightheartdevs Lightheartdevs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed: changes scoped to resources/dev/extensions-library only, no installer interaction. LGTM.

@Lightheartdevs
Copy link
Collaborator

Approved but has merge conflicts from other PRs that just landed. Please rebase against main and we'll merge. Thanks for the solid work on these! 🙏

@yasinBursali yasinBursali changed the base branch from resources/dev to main March 24, 2026 10:39
@yasinBursali yasinBursali dismissed Lightheartdevs’s stale review March 24, 2026 10:39

The base branch was changed.

@yasinBursali yasinBursali changed the base branch from main to resources/dev March 24, 2026 10:42
Three categories of fixes applied to service manifests:

A. Remove invalid gpu_backends values
   - Remove gpu_backends: [none] from service level where already corrected
     on resources/dev (crewai, baserow, paperless-ngx, weaviate,
     open-interpreter, milvus, gitea, label-studio)
   - Remove gpu_backends: [] from feature level in aider, chromadb,
     label-studio, piper-audio, sillytavern

B. Convert nested dict features to top-level array format (7 services)
   - bark, baserow, crewai, forge, frigate, paperless-ngx, weaviate
   - Each feature now includes all schema-required fields: id, name,
     description, icon, category, requirements, priority
   - GPU services include gpu_backends on every feature matching service level

C. Add missing compose_file field (4 services)
   - anythingllm, flowise, langflow, librechat

All changes validated against service-manifest.v1.json schema. YAML parsing
verified for all 33 service manifests.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@yasinBursali yasinBursali force-pushed the ext/fix-manifest-bulk-corrections branch from fa21c42 to c8a3262 Compare March 24, 2026 11:10
@yasinBursali
Copy link
Contributor Author

Rebased on latest resources/dev (af07063). Resolved librechat manifest conflict — kept depends_on: [] from resources/dev (per merged PR #555), added missing compose_file: compose.yaml. All 19 files, 1 clean commit.

@yasinBursali
Copy link
Contributor Author

Rebased on latest resources/dev (af07063). Resolved librechat conflict — kept depends_on: [] from resources/dev (per merged #555), added missing compose_file. This PR should merge before #566, #568, and #570 as they touch overlapping manifest files.

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