Skip to content

Commit 22cd982

Browse files
guyb1claude
andauthored
fix: align org API routes with project-level paths (#77)
Org app routes now mirror project routes prefixed with /org/: - /org/connections → /org/apps/connections - /org/app-config/:provider → /org/apps/:provider/config - /org/app-config/configured → /org/apps/configured - /org/app-config/:provider/toggle → /org/apps/:provider/config/toggle The API server has backward-compatible aliases for the old paths. Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 68cf02a commit 22cd982

2 files changed

Lines changed: 8 additions & 8 deletions

File tree

internal/api/org_apps.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ type ToggleInput struct {
2020
// ListConfiguredProviders returns providers that have org-level credentials configured.
2121
func (c *Client) ListConfiguredProviders(ctx context.Context) ([]any, error) {
2222
var providers []any
23-
if err := c.do(ctx, http.MethodGet, "/v1/org/app-config/configured", nil, &providers); err != nil {
23+
if err := c.do(ctx, http.MethodGet, "/v1/org/apps/configured", nil, &providers); err != nil {
2424
return nil, fmt.Errorf("listing configured providers: %w", err)
2525
}
2626
return providers, nil
@@ -29,7 +29,7 @@ func (c *Client) ListConfiguredProviders(ctx context.Context) ([]any, error) {
2929
// GetOrgAppConfig returns the app config for a provider at the org level.
3030
func (c *Client) GetOrgAppConfig(ctx context.Context, provider string) (*OrgAppConfig, error) {
3131
var config OrgAppConfig
32-
if err := c.do(ctx, http.MethodGet, "/v1/org/app-config/"+provider, nil, &config); err != nil {
32+
if err := c.do(ctx, http.MethodGet, "/v1/org/apps/"+provider+"/config", nil, &config); err != nil {
3333
return nil, fmt.Errorf("getting org app config: %w", err)
3434
}
3535
return &config, nil
@@ -38,15 +38,15 @@ func (c *Client) GetOrgAppConfig(ctx context.Context, provider string) (*OrgAppC
3838
// UpsertOrgAppConfig saves BYOC credentials for a provider at the org level.
3939
func (c *Client) UpsertOrgAppConfig(ctx context.Context, provider string, input ConfigAppInput) error {
4040
var resp SuccessResponse
41-
if err := c.do(ctx, http.MethodPost, "/v1/org/app-config/"+provider, input, &resp); err != nil {
41+
if err := c.do(ctx, http.MethodPost, "/v1/org/apps/"+provider+"/config", input, &resp); err != nil {
4242
return fmt.Errorf("configuring org app: %w", err)
4343
}
4444
return nil
4545
}
4646

4747
// DeleteOrgAppConfig removes BYOC credentials for a provider at the org level.
4848
func (c *Client) DeleteOrgAppConfig(ctx context.Context, provider string) error {
49-
if err := c.do(ctx, http.MethodDelete, "/v1/org/app-config/"+provider, nil, nil); err != nil {
49+
if err := c.do(ctx, http.MethodDelete, "/v1/org/apps/"+provider+"/config", nil, nil); err != nil {
5050
return fmt.Errorf("removing org app config: %w", err)
5151
}
5252
return nil
@@ -55,7 +55,7 @@ func (c *Client) DeleteOrgAppConfig(ctx context.Context, provider string) error
5555
// ToggleOrgAppConfig enables or disables an app config at the org level.
5656
func (c *Client) ToggleOrgAppConfig(ctx context.Context, provider string, enabled bool) error {
5757
var resp SuccessResponse
58-
if err := c.do(ctx, http.MethodPatch, "/v1/org/app-config/"+provider+"/toggle", ToggleInput{Enabled: enabled}, &resp); err != nil {
58+
if err := c.do(ctx, http.MethodPatch, "/v1/org/apps/"+provider+"/config/toggle", ToggleInput{Enabled: enabled}, &resp); err != nil {
5959
return fmt.Errorf("toggling org app config: %w", err)
6060
}
6161
return nil

internal/api/org_connections.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ type Connection struct {
1717
// ListOrgConnections returns all connections for the organization.
1818
func (c *Client) ListOrgConnections(ctx context.Context) ([]Connection, error) {
1919
var connections []Connection
20-
if err := c.do(ctx, http.MethodGet, "/v1/org/connections", nil, &connections); err != nil {
20+
if err := c.do(ctx, http.MethodGet, "/v1/org/apps/connections", nil, &connections); err != nil {
2121
return nil, fmt.Errorf("listing org connections: %w", err)
2222
}
2323
return connections, nil
@@ -26,15 +26,15 @@ func (c *Client) ListOrgConnections(ctx context.Context) ([]Connection, error) {
2626
// ListOrgConnectionsByProvider returns connections for a specific provider.
2727
func (c *Client) ListOrgConnectionsByProvider(ctx context.Context, provider string) ([]Connection, error) {
2828
var connections []Connection
29-
if err := c.do(ctx, http.MethodGet, "/v1/org/connections/"+provider, nil, &connections); err != nil {
29+
if err := c.do(ctx, http.MethodGet, "/v1/org/apps/connections/"+provider, nil, &connections); err != nil {
3030
return nil, fmt.Errorf("listing org connections for provider: %w", err)
3131
}
3232
return connections, nil
3333
}
3434

3535
// DeleteOrgConnection removes an org-scoped connection by ID.
3636
func (c *Client) DeleteOrgConnection(ctx context.Context, connectionID string) error {
37-
if err := c.do(ctx, http.MethodDelete, "/v1/org/connections/"+connectionID, nil, nil); err != nil {
37+
if err := c.do(ctx, http.MethodDelete, "/v1/org/apps/connections/"+connectionID, nil, nil); err != nil {
3838
return fmt.Errorf("deleting org connection: %w", err)
3939
}
4040
return nil

0 commit comments

Comments
 (0)