Skip to content

AB-1934: Add TS and Sync support#920

Merged
animeshjajoopostman merged 42 commits intodevelopfrom
feature/ts-support
Jan 28, 2026
Merged

AB-1934: Add TS and Sync support#920
animeshjajoopostman merged 42 commits intodevelopfrom
feature/ts-support

Conversation

@animeshjajoopostman
Copy link
Copy Markdown
Collaborator

@animeshjajoopostman animeshjajoopostman commented Dec 22, 2025

  • Add support for syncing collections with changes from specs.
  • Add TS support

JIRA: https://postmanlabs.atlassian.net/browse/AB-1934

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 22, 2025

integration test code coverage

Lines Statements Branches Functions
Coverage: 19%
19.57% (2100/10726) 13.35% (999/7479) 20.78% (248/1193)
Coverage Breakdown • (19%)
File% Stmts% Branch% Funcs% LinesUncovered Line #s
All files19.5713.3520.7819.87 
lib40.4439.3327.3940.74 
   options.js6245.9457.1463.26...50–462, 476–502
   schemaUtils.js47.6944.4738.5548.01...3247, 3281–5381
   schemapack.js27.1915.648.9227.51...18–420, 441–868
libV29.22009.31 
   index.js12.120012.1229–364
libV2/CollectionGeneration5.48005.54 
   requestMatchingUtils.js1.52001.5511–229, 246–385
   schemaUtils.js7.65007.72...2808, 2821–2881
   utils.js11.940012.129–51, 54–82, 92–242
   validationUtils.js3.11003.1498–2539, 2547–2792
test/integration100100100100 
   integration.test.js100100100100 
test/unit86.66508089.65 
   sanity.test.js86.66508089.6538–41

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 22, 2025

unit test code coverage

Lines Statements Branches Functions
Coverage: 88%
88.84% (5876/6614) 80.6% (3865/4795) 93.99% (861/916)
Coverage Breakdown • (88%)
File% Stmts% Branch% Funcs% LinesUncovered Line #s
All files88.8480.693.9988.95 
bin55.7964.8666.6656.61 
   openapi2postmanv2.js55.7964.8666.6656.61...86–287, 305–306
lib93.0286.5596.8893 
   options.js9472.9710095.91424, 502
   schemaUtils.js93.3987.3598.7293.31...5346–5348, 5372
   schemapack.js88.380.2794.6488.16...54, 770, 778–779
libV278.1568.8810077.94 
   index.js69.6953.7310069.69...18, 343, 347, 364
libV2/CollectionGeneration90.6382.0595.0190.66 
   requestMatchingUtils.js96.9491.1710096.89263–267, 284, 341
   schemaUtils.js93.3887.3499.0193.6...2543, 2692–2698
   utils.js95.5283.3394.7395.45112, 161, 172
   validationUtils.js86.2674.6191.4886.1...2725, 2762–2765

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds TypeScript support to the project by introducing type definitions and converting the main entry point to TypeScript. The changes enable better type safety and developer experience when using this OpenAPI to Postman converter library.

Key changes:

  • New TypeScript type definitions in types/index.ts covering all major interfaces
  • Conversion of index.js to index.ts with TypeScript annotations
  • Build pipeline setup with TypeScript compilation to dist/ directory

Reviewed changes

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

Show a summary per file
File Description
types/index.ts Comprehensive type definitions for converter inputs, options, and outputs
index.ts TypeScript conversion of main entry point with type annotations
package.json Added TypeScript tooling, build scripts, and updated main/types entry points
libV2/index.js Updated import paths to reference reorganized CollectionGeneration directory
libV2/CollectionGeneration/schemaUtils.js Added missing eslint-disable comment and fixed import paths with .js extensions
libV2/CollectionGeneration/validationUtils.js Updated import paths with .js extensions for relocated modules
test/unit/*.test.js Updated test imports to use compiled dist/index.js instead of source
test/system/*.test.js Updated test imports to use compiled dist/index.js
test/integration/integration.test.js Updated import to use compiled dist/index.js
scripts/test-lint.sh Removed index.js from eslint targets since it's now TypeScript

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

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

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


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

@AyushShri
Copy link
Copy Markdown
Collaborator

are we not creating index.d.ts for index.ts ?

@animeshjajoopostman animeshjajoopostman marked this pull request as ready for review January 27, 2026 10:42
@animeshjajoopostman animeshjajoopostman changed the title Add ts support Add ts and syncing support Jan 28, 2026
@animeshjajoopostman animeshjajoopostman changed the title Add ts and syncing support AB-1934: Add ts and syncing support Jan 28, 2026
@animeshjajoopostman animeshjajoopostman changed the title AB-1934: Add ts and syncing support AB-1934: Add TS and Sync support Jan 28, 2026
@animeshjajoopostman animeshjajoopostman merged commit 13b061b into develop Jan 28, 2026
12 checks passed
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.

3 participants