Skip to content

chore(sdk): TSSDK-57 remove processRunItem & processApplicationRun#79

Open
mk0x9 wants to merge 1 commit into
mainfrom
chore/TSSDK-57-remove-processRunItem-and-processApplicationRun
Open

chore(sdk): TSSDK-57 remove processRunItem & processApplicationRun#79
mk0x9 wants to merge 1 commit into
mainfrom
chore/TSSDK-57-remove-processRunItem-and-processApplicationRun

Conversation

@mk0x9

@mk0x9 mk0x9 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

TSSDK-57

PlatformSDKHttp.getRun(), listApplicationRuns(), and listRunResults() no longer auto-enrich responses — they return the raw RunReadResponse / ItemResultReadResponse from SAMIA as is.

  • consumers derive progress/status/can_download on demand via the already-exported getRunProgress, getRunStatus, canDownloadRunItems, getItemStatus, canDownloadItem

@tobi-ax tobi-ax left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🪓

return responses from SAMIA as is, instead export helper functions for
querying progress, status, et cetera
@mk0x9 mk0x9 force-pushed the chore/TSSDK-57-remove-processRunItem-and-processApplicationRun branch from 9cc4419 to 385e2f2 Compare July 7, 2026 11:58
@sonarqubecloud

sonarqubecloud Bot commented Jul 7, 2026

Copy link
Copy Markdown

@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #79      +/-   ##
==========================================
- Coverage   97.00%   96.96%   -0.04%     
==========================================
  Files          14       14              
  Lines        1400     1383      -17     
  Branches      211      211              
==========================================
- Hits         1358     1341      -17     
  Misses         42       42              
Flag Coverage Δ
unittests 96.96% <100.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
packages/sdk/src/entities/application-run/index.ts 100.00% <100.00%> (ø)
packages/sdk/src/entities/run-item/index.ts 100.00% <100.00%> (ø)
packages/sdk/src/platform-sdk.ts 98.85% <100.00%> (-0.01%) ⬇️

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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