3847eb2036
This repo had no version control prior to this commit. The import is a
straight snapshot of the working tree at 2026-05-03; the deployed
binary on fihelvop01 was being rebuilt from this source via `make
build` + scp into place, with no upstream review path.
The snapshot already includes one in-flight fix made on 2026-05-03 to
internal/service/persona.go:GetSelfModel — the handler queried
`source` and `strength` columns plus an `is_active = true` filter on
persona.persona_commitments, none of which exist on that table (its
shape is session-bound commitments with `status`, `commitment_meta`,
etc.). The query returned a 500 every time SynapseHub bootstrapped a
persona's self-model, dropping the IdentityConstraints / Commitments /
ConscienceStandards layer from the assembled prompt. The patched
query reads existing columns only (commitment_text, commitment_type),
filters on `status='active'`, and synthesises Source="learned" /
Strength=1.0 to keep the SelfModel response shape stable for callers.
Verified live: `GET /api/v1/personas/70f7cfd9-.../self-model` now
returns 200 with `{identityConstraints:[],commitments:[],
conscienceStandards:[]}` instead of 500.
Future changes go through PRs against this repo — no more bin-only
deploys.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
68 lines
1.6 KiB
Go
68 lines
1.6 KiB
Go
package client
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
"time"
|
|
|
|
"github.com/jackc/pgx/v5/pgxpool"
|
|
"github.com/rs/zerolog"
|
|
|
|
"github.com/gosec/gsc-ops-api/internal/config"
|
|
)
|
|
|
|
// CardDAVClient wraps a pgx connection pool for the sabredav database
|
|
type CardDAVClient struct {
|
|
pool *pgxpool.Pool
|
|
logger zerolog.Logger
|
|
}
|
|
|
|
// NewCardDAVClient creates a new CardDAV database client
|
|
func NewCardDAVClient(cfg config.CardDAVConfig, dsn string, logger zerolog.Logger) (*CardDAVClient, error) {
|
|
poolConfig, err := pgxpool.ParseConfig(dsn)
|
|
if err != nil {
|
|
return nil, fmt.Errorf("failed to parse carddav database config: %w", err)
|
|
}
|
|
|
|
poolConfig.MaxConns = 10
|
|
poolConfig.MinConns = 2
|
|
poolConfig.MaxConnLifetime = 1 * time.Hour
|
|
poolConfig.MaxConnIdleTime = 30 * time.Minute
|
|
poolConfig.HealthCheckPeriod = 1 * time.Minute
|
|
|
|
ctx, cancel := context.WithTimeout(context.Background(), 10*time.Second)
|
|
defer cancel()
|
|
|
|
pool, err := pgxpool.NewWithConfig(ctx, poolConfig)
|
|
if err != nil {
|
|
return nil, fmt.Errorf("failed to create carddav connection pool: %w", err)
|
|
}
|
|
|
|
if err := pool.Ping(ctx); err != nil {
|
|
pool.Close()
|
|
return nil, fmt.Errorf("failed to ping carddav database: %w", err)
|
|
}
|
|
|
|
return &CardDAVClient{
|
|
pool: pool,
|
|
logger: logger.With().Str("client", "carddav").Logger(),
|
|
}, nil
|
|
}
|
|
|
|
// Pool returns the underlying connection pool
|
|
func (c *CardDAVClient) Pool() *pgxpool.Pool {
|
|
return c.pool
|
|
}
|
|
|
|
// Health checks the database connection
|
|
func (c *CardDAVClient) Health(ctx context.Context) error {
|
|
return c.pool.Ping(ctx)
|
|
}
|
|
|
|
// Close closes the connection pool
|
|
func (c *CardDAVClient) Close() {
|
|
if c.pool != nil {
|
|
c.pool.Close()
|
|
}
|
|
}
|