Skip to content

feat: config full endpoint#146

Merged
CptSchnitz merged 10 commits intomasterfrom
config-full-endpoint
Feb 10, 2026
Merged

feat: config full endpoint#146
CptSchnitz merged 10 commits intomasterfrom
config-full-endpoint

Conversation

@CptSchnitz
Copy link
Collaborator

No description provided.

Copy link
Contributor

@ronenkapelian ronenkapelian left a comment

Choose a reason for hiding this comment

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

  1. Migration 0004 failed on re-run
    error: column "hash" of relation "config" already exists

ALTER TABLE config ADD COLUMN IF NOT EXISTS "hash" text;

  1. fix merge conflict

…refactoring

- Add comprehensive metadata endpoint for config inspector page
- Implement getFullConfigMetadata() with caching (5min TTL, async-cache-dedupe)
- Return metadata, versions, dependencies, env vars, and stats in single call
- Add 15 integration tests (happy/bad/sad paths, caching, complex scenarios)
- Refactor OpenAPI schemas to use configBaseMetadata for DRY
- Fix all ESLint errors (extracted magic numbers, fixed types)
- All 168 tests passing
- Add explanatory comment about OpenAPI 3.0 limitation workaround
- Suppress member-ordering warnings (methods logically grouped by feature)
- Remove unused imports (gt, Lock) and unused variable (response2)
- All 168 tests passing, 0 ESLint warnings/errors
Copy link
Contributor

@ronenkapelian ronenkapelian left a comment

Choose a reason for hiding this comment

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

:(

@CptSchnitz CptSchnitz merged commit f8b67ea into master Feb 10, 2026
4 of 5 checks passed
@CptSchnitz CptSchnitz deleted the config-full-endpoint branch February 10, 2026 14:04
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