Skip to content

Conversation

@evens-stone
Copy link
Contributor

Description

Please provide a short summary of the changes you have made in this pull request. Include any context or background that might be helpful to reviewers.

Related Issue

Fixes # (issue number)

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactoring
  • Other (please describe):

Checklist (Preconditions for Code Review)

  • Lint your code to ensure it adheres to the project's style guidelines.
  • Run all existing and new unit tests locally to verify correctness.
  • Ensure code changes are covered with appropriate tests (unit, integration, etc.).
  • Update all dependencies to their latest compatible versions.
  • Update documentation where necessary to reflect the changes.
  • Resolve all linting and compilation errors.
  • Follow SOLID principles and other best practices in the code.
  • Review the code for potential security vulnerabilities.
  • Add comments for complex or hard-to-understand parts of the code.
  • Ensure the PR is appropriately scoped (small, focused, and easy to review).
  • Ensure commit messages adhere to the Conventional commit message guidelines.
  • Add tests for any new features or bug fixes.
  • Clearly indicate if the pull request introduces any breaking changes.

Screenshots (if applicable)

Please add screenshots to help explain your changes if applicable.

Additional Information

Please provide any additional information that might be helpful during the review process.

@evens-stone evens-stone requested a review from Copilot June 16, 2025 05:04
@evens-stone evens-stone self-assigned this Jun 16, 2025
@evens-stone evens-stone added the enhancement New feature or request label Jun 16, 2025
Copy link

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 pull request introduces a browser-safe fallback entry with no-op modules for frontend usage while maintaining the existing Node.js behavior.

  • Excludes browser-specific files from the main TypeDoc documentation.
  • Adds browser-specific decorator and blueprint stubs to support frontend environments.
  • Updates tests, Rollup configuration, and package exports to distinguish browser and Node.js bundles.

Reviewed Changes

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

Show a summary per file
File Description
typedoc.json Excluded "src/browser/**/*" to prevent browser modules in docs.
tests/decorators/AwsLambdaHttp.spec.ts Added tests for browser-specific decorator; updated mocking usage.
src/browser/options/NodeHttpAdapterBlueprint.ts Introduced a no-op blueprint stub for browser environments.
src/browser/decorators/AwsLambdaHttp.ts Added a stubbed browser decorator for AWS Lambda HTTP.
rollup.config.mjs Updated input patterns for proper separation of browser and Node files.
package.json Added browser export field and updated Node engine requirement.

@sonarqubecloud
Copy link

@evens-stone evens-stone merged commit 19e7c12 into main Jun 16, 2025
5 checks passed
@evens-stone evens-stone deleted the fixup branch June 16, 2025 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants