Skip to content

#1035, #1036: Single-folder vs Subfolder Detection #1048

Open
ethanluc7 wants to merge 2 commits into3.1from
1035-handle-multiple-folder-configs
Open

#1035, #1036: Single-folder vs Subfolder Detection #1048
ethanluc7 wants to merge 2 commits into3.1from
1035-handle-multiple-folder-configs

Conversation

@ethanluc7
Copy link
Copy Markdown

@ethanluc7 ethanluc7 commented Apr 1, 2026

Related to issue: #1035, #1036

Summary

Extended buildCatalogsFromFolder function to support multi-catalog configuration by detecting whether the config path contains JSON files directly (single-catalog) or subdirectories (multi-catalog).

Description of Changes

  • Extended buildCatalogsFromFolder to detect JSON files vs subdirectories in the config path
  • Single-catalog mode (backward-compatible): reads JSONs directly from the config folder
  • Multi-catalog mode: iterates subdirectories, each producing a separate catalog entry

Readiness Checklist

  • Self Review
    • I have performed a self review of code
    • I have run the application locally and manually tested the feature
    • I have checked all updates to correct typos and misspellings
  • Formatting
    • Code follows the project style guide
    • Automated code formatters (ie. Prettier) have been run
  • Local Testing
    • Successfully built all packages locally
    • Successfully ran all test suites, all unit and integration tests pass
  • Updated Tests
    • Unit and integration tests have been added that describe the bug that was fixed or the features that were added
  • Documentation
    • All new environment variables added to .env.schema file and documented in the README
    • All changes to server HTTP endpoints have open-api documentation
    • All new functions exported from their module have TSDoc comment documentation

@ethanluc7 ethanluc7 changed the title feat: allow support for multiple catalogs #1035, #1036: Single-folder vs Subfolder Detection Apr 2, 2026
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.

1 participant