docs(kb): GDS/NDC per-carrier×vendor×transaction capability matrix (#142) - #165
Merged
Merged
Conversation
) Add authoritative KB matrix for Agent 3.1 so routing is keyed by (carrier, vendor, transaction), not a single airline→channel map. - Res 787 framed as Shop/Order/Change process standard (not parity) - Sabre Offer/Order REST surface cited from public NDC guide - Anonymized rows from adapter docs; shop≠servicing for Duffel + TripPro - Matrix loader + router consumption; no invented NDC 21.3 defaults Co-authored-by: telivity-otaip <telivity-otaip@users.noreply.github.com>
Reflect transaction_type, vendor, plating_carrier, and capability_matrix on Agent 3.1 schema after #142 KB matrix work. Co-authored-by: telivity-otaip <telivity-otaip@users.noreply.github.com>
telivity-otaip
marked this pull request as ready for review
August 24, 2026 08:40
Co-authored-by: telivity-otaip <telivity-otaip@users.noreply.github.com>
|
CI note: The first CI run failed only on Subsequent agent pushes did not re-trigger Actions (token cannot re-run workflows). Please Re-run failed jobs on the first CI run, or push an empty commit from a maintainer account, so CI can validate HEAD. Local verification already green:
|
Merge origin/main into cursor/gds-ndc-capability-matrix-2500. Only conflict was generated docs/agent-map.html (both branches regenerated the manifest). Regenerated agents.manifest.json and agent-map.html to combine main's agent updates with GdsNdcRouter matrix input fields. Co-authored-by: telivity-otaip <telivity-otaip@users.noreply.github.com>
Main advanced with #163 (BSP HOT multi-currency). Only conflict was generated docs/agent-map.html; regenerated manifest to combine both sides. Co-authored-by: telivity-otaip <telivity-otaip@users.noreply.github.com>
GdsNdcRouter contract schema requires transaction_type since per- transaction routing landed. The Sprint A happy-path pipeline omitted it, causing schema validation to fail at step 3.1. Co-authored-by: telivity-otaip <telivity-otaip@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Closes #142. Adds an authoritative GDS/NDC capability matrix so Agent 3.1 routes by
(carrier, vendor, transaction)— never a single airline→channel map.Type of change
Key points
Shop,OrderCreate,OrderChange,OrderCancel,Servicing,Groups,Corporate), channel, NDC version notes, fallback, source, confidence.ANON-NDC-AGG(Duffel) andANON-AGG-DUAL(TripPro).21.3— unknown stays unknown; NDC format stub no longer defaults to 21.3.vendor,plating_carrier,capability_matrix, plus existingtransaction_type/ overrides. Loader incapability-matrix.ts.Domain impact
Hardens Agent 3.1 against per-airline channel rationalization (CLAUDE.md guards). Production carrier enablement for Sabre Offer/Order endpoints remains
DOMAIN_QUESTION/unknownuntil evidenced.Test plan
vitest run packages/agents/booking/src/gds-ndc-router— 60 passed (includes matrix shop≠servicing + no invented version)pnpm gen:manifest --check— up to date