feat: Add ignore_archived_cards field in create card transactions, summary, reconciliation summary and reconcile apis

[madangopal122]

Description

Please add PR description here, add screenshots if needed

Clickup

Please add link here

[Copilot]

Pull request overview

Adds an ignore_archived_cards request field to statement-processing input schemas so API consumers can control whether archived corporate cards are skipped or unarchived during statement processing (create transactions / summary / reconciliation).

Changes:

• Added ignore_archived_cards to card_transaction_from_statement_in, summary_from_statement_in, reconciliation_summary_from_statement_in, and reconcile_from_statement_in.
• Reworked statement_archived_cards_in from an allOf composition to an explicit object schema (likely to avoid inheriting new fields).

Comments suppressed due to low confidence (3)

src/components/schemas/statements.yaml:496

• The new ignore_archived_cards field description mentions "creating card transactions", but this schema is used for the statement summary API. The description should reflect statement summary/reconciliation processing rather than transaction creation.

      description: |
        If true, then the archived cards will be ignored while creating card transactions from the statement.
        If false, then the archived cards will be unarchived and transactions will be created for them.

src/components/schemas/statements.yaml:770

• The ignore_archived_cards field description refers to "creating card transactions", but this schema is used by the reconciliation summary API. Update the description to be accurate for reconciliation processing.

      description: |
        If true, then the archived cards will be ignored while creating card transactions from the statement.
        If false, then the archived cards will be unarchived and transactions will be created for them.

src/components/schemas/statements.yaml:907

• The ignore_archived_cards field description refers to "creating card transactions", but this schema is used by the reconcile API (background reconciliation). Update the description to be accurate for reconciliation processing.

      description: |
        If true, then the archived cards will be ignored while creating card transactions from the statement.
        If false, then the archived cards will be unarchived and transactions will be created for them.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

src/components/schemas/statements.yaml:1476

• statement_archived_cards_in no longer defines a top-level required list, so fields like id, statement_mapping, and card_mapping become optional in the schema. Previously these were required (via the earlier allOf composition), so this is a breaking/incorrect contract change.

    card_mapping:
      $ref: '#/card_mapping_in'

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (3)

src/components/schemas/statements.yaml:770

• reconciliation_summary_from_statement_in.ignore_archived_cards is missing an example, unlike the same field in other statement-processing inputs. Adding one keeps the schema docs consistent and easier to understand.

    ignore_archived_cards:
      type: boolean
      nullable: true
      description: |
        Controls how transactions associated with archived cards are handled during statement processing:

src/components/schemas/statements.yaml:907

• reconcile_from_statement_in.ignore_archived_cards is missing an example, but the same field elsewhere includes one. Adding an explicit example helps make the generated docs consistent.

    ignore_archived_cards:
      type: boolean
      nullable: true
      description: |
        Controls how transactions associated with archived cards are handled during statement processing:

src/components/schemas/statements.yaml:1371

• statement_archived_cards_in was expanded from an allOf composition into a fully inlined schema that largely duplicates summary_from_statement_in (same id, statement_mapping, ignored_line_item_ids, etc.). This increases the chance of the two inputs drifting over time. Consider reverting to an allOf with $ref: '#/summary_from_statement_in' and only layering additional properties (start_date, end_date) if needed.

statement_archived_cards_in:
  type: object
  properties:
    id:
      allOf:

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

src/components/schemas/statements.yaml:1479

• statement_archived_cards_in previously required card_mapping (it was required under statement_mapping via the earlier allOf schema). After moving card_mapping to a top-level property, it is no longer listed as required, which changes the request contract and likely makes the request underspecified for mapping statement card numbers to corporate cards. If card_mapping is still required for this endpoint, add it back to the required list.

    card_mapping:
      $ref: '#/card_mapping_in'
  required:
    - id
    - statement_mapping