Skip to content

feat: add TypeScript types for bigint.js#21

Merged
ChALkeR merged 10 commits intomainfrom
copilot/add-types-for-bigint-js
Jan 22, 2026
Merged

feat: add TypeScript types for bigint.js#21
ChALkeR merged 10 commits intomainfrom
copilot/add-types-for-bigint-js

Conversation

Copy link
Contributor

Copilot AI commented Jan 22, 2026

  • Create bigint.d.ts type definition file
  • Add types export to package.json exports for ./bigint.js
  • Add bigint.d.ts to package.json files list
  • Run tests to verify types work correctly
  • Address code review feedback on type overloads
  • Simplify type definitions for better readability
  • Run security checks (0 issues found)
  • Document endianness in type definitions
  • Sync README.md with type definitions
  • Rename parameter for consistency
  • Complete task
Original prompt

This section details on the original issue you should resolve

<issue_title>Types: bigint.js</issue_title>
<issue_description>Add types for /bigint.js</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: ChALkeR <291301+ChALkeR@users.noreply.github.com>
@ChALkeR ChALkeR mentioned this pull request Jan 22, 2026
17 tasks
Copilot AI and others added 2 commits January 22, 2026 08:35
Co-authored-by: ChALkeR <291301+ChALkeR@users.noreply.github.com>
Co-authored-by: ChALkeR <291301+ChALkeR@users.noreply.github.com>
Copilot AI changed the title [WIP] Add types for bigint.js Add TypeScript types for bigint.js Jan 22, 2026
Copilot AI requested a review from ChALkeR January 22, 2026 08:39
Co-authored-by: ChALkeR <291301+ChALkeR@users.noreply.github.com>
Copy link
Collaborator

@ChALkeR ChALkeR left a comment

Choose a reason for hiding this comment

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

@copilot Sync README.md @exodus/bytes/bigint.js section with this.

Co-authored-by: ChALkeR <291301+ChALkeR@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 22, 2026

@copilot Sync README.md @exodus/bytes/bigint.js section with this.

Done. Synced the README.md bigint.js section with the type definitions, including descriptions, big-endian format documentation, and error conditions (43f57dc).

Copilot AI requested a review from ChALkeR January 22, 2026 08:52
Co-authored-by: ChALkeR <291301+ChALkeR@users.noreply.github.com>
Copilot AI requested a review from ChALkeR January 22, 2026 08:54
@ChALkeR ChALkeR marked this pull request as ready for review January 22, 2026 08:57
@ChALkeR ChALkeR requested a review from Copilot January 22, 2026 08:58
@ChALkeR ChALkeR changed the title Add TypeScript types for bigint.js feat: add TypeScript types for bigint.js Jan 22, 2026
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

Adds first-class TypeScript typings for @exodus/bytes/bigint.js, and wires them into package exports/documentation.

Changes:

  • Added bigint.d.ts with overloads for fromBigInt / toBigInt and documented big-endian behavior
  • Updated package.json files + exports to publish and expose the new typings for ./bigint.js
  • Updated README section for @exodus/bytes/bigint.js to match the typed API and endianness docs

Reviewed changes

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

File Description
package.json Publishes bigint.d.ts and exposes it via exports["./bigint.js"].types
bigint.d.ts Introduces TypeScript definitions for BigInt ↔ bytes conversions
README.md Expands module docs to reflect behavior and endianness

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

@ChALkeR ChALkeR merged commit fb1a7eb into main Jan 22, 2026
36 checks passed
Copilot AI added a commit that referenced this pull request Jan 22, 2026
@ChALkeR ChALkeR deleted the copilot/add-types-for-bigint-js branch January 22, 2026 20:32
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.

Types: bigint.js

2 participants