Skip to content

Commit b7b9441

Browse files
JAORMXclaude
andcommitted
Add toolhive-registry to RFC
toolhive-registry heavily imports from toolhive: - pkg/logger - pkg/permissions - pkg/registry/* - pkg/container/verifier Added to code sharing table, implementation tracking, and consumer migration plan. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 0616c79 commit b7b9441

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

rfcs/THV-0032-toolhive-core-shared-library.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ Additionally, TypeScript projects exist (toolhive-studio, toolhive-cloud-ui) but
2525
| Project | Imports from toolhive | Pattern Divergence |
2626
|---------|----------------------|-------------------|
2727
| **dockyard** | `pkg/logger`, `pkg/container/images`, `pkg/runner` | Aligned - uses toolhive utilities |
28+
| **toolhive-registry** | `pkg/logger`, `pkg/permissions`, `pkg/registry/*`, `pkg/container/verifier` | Aligned - heavily uses toolhive utilities |
2829
| **toolhive-registry-server** | Only `pkg/versions` + `pkg/registry` types | Uses stdlib `log/slog` instead of toolhive's zap logger; custom error handling |
2930

3031
**Current limitations:**
@@ -387,6 +388,7 @@ import "github.com/stacklok/toolhive-core/logger"
387388

388389
- Update `toolhive` to import from `toolhive-core`
389390
- Update `dockyard` to import from `toolhive-core` (currently imports `pkg/logger`, `pkg/container/images`, `pkg/runner`)
391+
- Update `toolhive-registry` to import from `toolhive-core` (currently imports `pkg/logger`, `pkg/permissions`, `pkg/registry/*`)
390392
- Offer `toolhive-registry-server` the option to adopt toolhive-core (may choose to continue with slog)
391393
- Deprecate original packages with forwarding imports
392394
- Monitor for issues and release patches as needed
@@ -475,4 +477,5 @@ import "github.com/stacklok/toolhive-core/logger"
475477
| toolhive-core | N/A | Not started |
476478
| toolhive | N/A | Not started |
477479
| dockyard | N/A | Not started |
480+
| toolhive-registry | N/A | Not started |
478481
| toolhive-registry-server | N/A | Optional - may continue using slog |

0 commit comments

Comments
 (0)