Skip to content

♻️ standardize node built-in imports with node: prefix - #492

Merged
ianardee merged 1 commit into
mainfrom
import-cleanup
Sep 18, 2026
Merged

ianardee merged 1 commit into
mainfrom
import-cleanup

Conversation

@ianardee

Copy link
Copy Markdown
Collaborator

Description

Types of changes

  • Docs change / refactoring / dependency upgrade
  • 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 change)
  • Requires a change to the official Guide documentation.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The reviewed changes have no unresolved blocking issues.

Pull request overview

Standardizes Node.js built-in imports with the node: prefix across the SDK and tests, alongside lint, type-checking, documentation, and minor correctness updates.

Changes:

  • Standardizes built-in imports across source and test files.
  • Enables full library type checking and stricter JSDoc validation.
  • Preserves DataSchema instances and updates related tests and error handling.
File summaries
File Reviewed changes
tsconfig.json Enables full library type checking.
tests/v2/search/ragDocumentSearch.spec.ts Standardizes path import.
tests/v2/search/modelSearch.spec.ts Standardizes path import.
tests/v2/product/split.spec.ts Standardizes path import.
tests/v2/product/ocr.spec.ts Standardizes path import.
tests/v2/product/extraction/ragDocuments.spec.ts Updates imports and helper placement.
tests/v2/product/extraction/ragDocuments.integration.ts Standardizes path import.
tests/v2/product/extraction/extractionParameter.spec.ts Standardizes imports.
tests/v2/product/extraction/extraction.spec.ts Updates imports and numeric assertions.
tests/v2/product/crop.spec.ts Standardizes imports.
tests/v2/product/classification.spec.ts Standardizes path import.
tests/v2/parsing/localResponse.spec.ts Standardizes path import.
tests/v2/parsing/job.spec.ts Standardizes path import.
tests/v2/parsing/failedInferenceResponse.spec.ts Standardizes path import.
tests/v2/parsing/errorResponse.spec.ts Standardizes path import.
tests/v2/fileOperations/split.spec.ts Standardizes path import.
tests/v2/fileOperations/crop.spec.ts Standardizes path import.
tests/v2/client/client.spec.ts Standardizes path import.
tests/v1/workflows/workflow.spec.ts Standardizes path and filesystem imports.
tests/v1/workflows/workflow.integration.ts Standardizes path import.
tests/v1/product/us/bankCheck/bankCheckV1.spec.ts Standardizes built-in imports.
tests/v1/product/resume/resumeV1.spec.ts Standardizes built-in imports.
tests/v1/product/receipt/receiptV5.spec.ts Standardizes built-in imports.
tests/v1/product/passport/passportV1.spec.ts Standardizes built-in imports.
tests/v1/product/multiReceiptsDetector/multiReceiptsDetectorV1.spec.ts Standardizes built-in imports.
tests/v1/product/invoiceSplitter/invoiceSplitterV1.spec.ts Standardizes built-in imports.
tests/v1/product/invoice/invoiceV4.spec.ts Standardizes built-in imports.
tests/v1/product/internationalId/internationalIdV2.spec.ts Standardizes built-in imports.
tests/v1/product/generated/generatedV1.spec.ts Standardizes built-in imports.
tests/v1/product/fr/idCard/idCardV2.spec.ts Standardizes built-in imports.
tests/v1/product/fr/carteGrise/carteGriseV1.spec.ts Standardizes built-in imports.
tests/v1/product/fr/bankAccountDetails/bankAccountDetailsV2.spec.ts Standardizes built-in imports.
tests/v1/product/fr/bankAccountDetails/bankAccountDetailsV1.spec.ts Standardizes built-in imports.
tests/v1/product/financialDocument/financialDocumentV1.spec.ts Standardizes built-in imports.
tests/v1/product/driverLicense/driverLicenseV1.spec.ts Standardizes built-in imports.
tests/v1/product/cropper/cropperV1.spec.ts Standardizes built-in imports.
tests/v1/product/barcodeReader/barcodeReaderV1.spec.ts Standardizes built-in imports.
tests/v1/parsing/localResponse.spec.ts Standardizes path import.
tests/v1/input/sources.integration.ts Standardizes built-in imports.
tests/v1/extras/ocr.spec.ts Standardizes path import.
tests/v1/extras/fullTextOcr.spec.ts Standardizes path import.
tests/v1/extras/extras.integration.ts Standardizes path import.
tests/v1/extraction/multiReceipts.spec.ts Standardizes path import.
tests/v1/extraction/multiReceipts.integration.ts Standardizes path import.
tests/v1/extraction/invoiceSplitter.spec.ts Standardizes path import.
tests/v1/extraction/invoiceSplitter.integration.ts Standardizes built-in imports.
tests/v1/api/response.spec.ts Standardizes path import.
tests/v1/api/feedbackResponse.spec.ts Standardizes path import.
tests/v1/api/endpoint.spec.ts Standardizes path import.
tests/v1/api/asyncResponse.spec.ts Standardizes path import.
tests/pdf/pdfTypes.spec.ts Standardizes path import.
tests/pdf/pdfOperation.spec.ts Standardizes built-in imports.
tests/input/sources.spec.ts Updates imports and stream typing.
tests/input/pageOperations.spec.ts Reorders built-in imports.
tests/input/compression.spec.ts Updates imports and PDF text assertions.
tests/index.ts Standardizes URL and path imports.
src/v2/product/extraction/ragDocuments/params/ragDocumentAnnotationParameters.ts Uses a more specific error type.
src/v2/product/extraction/params/extractionParameters.ts Preserves supplied DataSchema instances.
src/v2/parsing/inference/field/simpleField.ts Reformats scalar value initialization.
src/v2/http/mindeeApiV2.ts Reorders imports.
src/v2/client.ts Applies import and formatting cleanup.
src/v2/cli.ts Standardizes console import.
src/v1/parsing/standard/field.ts Fixes JSDoc formatting.
src/v1/parsing/generated/generatedObject.ts Fixes JSDoc formatting.
src/v1/parsing/common/feedback/feedbackResponse.ts Uses the configured source alias.
src/v1/parsing/common/extras/cropperExtra.ts Uses the configured source alias.
src/v1/http/workflowEndpoint.ts Standardizes URL import.
src/v1/http/endpoint.ts Standardizes URL import.
src/v1/http/apiSettingsV1.ts Cleans up formatting.
src/v1/extraction/multiReceiptsExtractor/multiReceiptsExtractor.ts Reorders imports.
src/v1/extraction/invoiceSplitterExtractor/invoiceSplitterExtractor.ts Reorders imports.
src/v1/client.ts Applies import cleanup.
src/v1/cli.ts Standardizes console import.
src/pdf/pdfUtils.ts Reorders imports.
src/pdf/pdfOperation.ts Reorders imports.
src/pdf/pdfExtractor.ts Standardizes path import.
src/pdf/extractedPdf.ts Standardizes filesystem imports.
src/parsing/baseLocalResponse.ts Standardizes buffer and crypto imports.
src/input/urlInput.ts Standardizes built-in imports.
src/input/streamInput.ts Standardizes stream and buffer imports.
src/input/pathInput.ts Standardizes path and filesystem imports.
src/input/localInputSource.ts Standardizes buffer import.
src/input/inputSource.ts Adds explicit buffer import.
src/input/bytesInput.ts Adds explicit buffer import and reformats documentation.
src/input/bufferInput.ts Adds explicit buffer import.
src/image/imageExtractor.ts Reorders imports.
src/image/imageCompressor.ts Adds explicit buffer import.
src/image/extractedImage.ts Standardizes filesystem imports.
src/http/dispatcher.ts Applies import cleanup.
src/http/baseSettings.ts Standardizes OS import.
src/http/apiCore.ts Reorders imports.
package.json Reorders development dependencies.
eslint.config.mjs Enables JSDoc prefix validation and formatting cleanup.
Review details
  • Files reviewed: 90/93 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

No unresolved review comments were identified, and the supplied assessments indicate approval readiness.

Review details
  • Files reviewed: 91/94 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@ianardee
ianardee merged commit a68b051 into main Sep 18, 2026
24 checks passed
@ianardee
ianardee deleted the import-cleanup branch September 18, 2026 16:18
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