Skip to content

fix(agentd): use authoritative trust scores in production#95

Merged
EfeDurmaz16 merged 1 commit into
mainfrom
fides-agentd-authoritative-trust-scores
May 6, 2026
Merged

fix(agentd): use authoritative trust scores in production#95
EfeDurmaz16 merged 1 commit into
mainfrom
fides-agentd-authoritative-trust-scores

Conversation

@EfeDurmaz16

Copy link
Copy Markdown
Owner

Summary

  • make production /v1/authorize resolve reputation and capability scores from trust-graph
  • keep request-body score fields only for non-production simulation
  • fail closed with 503 when production score lookup is unavailable
  • document production semantics for authorization score inputs

Verification

  • pnpm --filter @fides/agentd test
  • pnpm --filter @fides/agentd lint
  • pnpm --filter @fides/agentd build
  • git diff --check

@chatgpt-codex-connector

Copy link
Copy Markdown

💡 Codex Review

const response = await fetch(url)

P2 Badge Add a timeout to production score fetches

When trust-graph accepts the connection but stops responding, these production authorization lookups wait on fetch instead of failing closed, so /v1/authorize can hang for a long time rather than returning the intended 503 denial. This affects the new production-only score path; adding an AbortController timeout like the health probe uses would make the failure mode deterministic.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@EfeDurmaz16 EfeDurmaz16 force-pushed the fides-agentd-authoritative-trust-scores branch 2 times, most recently from 0e49a6a to 184e767 Compare May 6, 2026 17:24
@EfeDurmaz16 EfeDurmaz16 force-pushed the fides-agentd-authoritative-trust-scores branch from 184e767 to 06d9725 Compare May 6, 2026 17:31
@EfeDurmaz16 EfeDurmaz16 merged commit 06a1181 into main May 6, 2026
9 checks passed
@EfeDurmaz16 EfeDurmaz16 deleted the fides-agentd-authoritative-trust-scores branch May 6, 2026 17:36
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