Skip to content

docs(api): expose tags on account and operation responses#17

Merged
marioalvial merged 5 commits intomainfrom
docs/tags-on-account-and-operation
May 4, 2026
Merged

docs(api): expose tags on account and operation responses#17
marioalvial merged 5 commits intomainfrom
docs/tags-on-account-and-operation

Conversation

@marioalvial
Copy link
Copy Markdown
Contributor

Description

Documents the tags field on account and operation API responses. Tags are system-managed labels assigned by Trace based on routing, compliance, and operational criteria; clients cannot create, modify, or remove them via the API.

The wire shape is an array of objects with a nullable key and a required value, matching the live TagResponse in fx-account. Single-value labels (e.g., hidden) carry key: null; keyed tags (e.g., psp=Amazon, compliance-tier=high) populate both fields.

Account-side tags are already live in fx-account. Operation-side tags are aspirational — they ship in the spec ahead of fx-payment implementation per the project's aspirational-docs pattern.

Key Changes

  • Enriched TagResponse schema in apis/fx-account/openapi.yml with description, per-property descriptions, and realistic examples.
  • Added description and a 3-item array example to AccountResponse.tags.
  • Mirrored TagResponse into apis/fx-payment/openapi.yml.
  • Added required tags field to OperationResponse with description noting inheritance from the account.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Unit Test
  • Integration Test

Validated locally with mint validate (passes) and mint broken-links (passes).

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

Backend follow-up

fx-payment needs to catch up to the documented contract:

  1. Tag shared domain (or import from fx-account)
  2. tags: List<Tag> on Operation and OperationApiResponse
  3. Tag propagation logic when an operation is created against an account

@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 4, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
tracefinance 🟢 Ready View Preview May 4, 2026, 4:13 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

Backfills the tags array (previously missing) in the pixWithdrawal,
brlToUsdSwap, and pixDeposit named examples so the rendered response
panel matches the OperationResponse schema, which already lists tags
as required.
Mirrors the AccountResponse/OperationResponse pattern. Introduces
TagEvent in fx-webhook to keep the *Event suffix convention. Documented
ahead of the Kotlin OperationEvent class, which will need a matching
field on the wire.
Tags is optional on OperationResponse and OperationEvent — the live
OperationApiResponse and OperationEvent Kotlin classes don't yet carry
the field, so leaving required would force strict consumers to fail
against today's wire payloads. AccountResponse keeps tags required
since the live AccountResponse Kotlin class already exposes it.

Trims the Filters parameter override on /api/reports to drop the LHS
Brackets preamble already covered by the shared Filters description,
leaving only the report-specific examples and a pointer to the
Filtering guide.
@marioalvial marioalvial merged commit 7e332a9 into main May 4, 2026
9 checks passed
@marioalvial marioalvial deleted the docs/tags-on-account-and-operation branch May 4, 2026 21:56
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