Skip to content

chore: migrate code from googleapis/proto3-json-serializer-nodejs#86

Merged
miguelvelezsa merged 86 commits intomainfrom
proto3-json-serializer-nodejs-migration
Oct 10, 2025
Merged

chore: migrate code from googleapis/proto3-json-serializer-nodejs#86
miguelvelezsa merged 86 commits intomainfrom
proto3-json-serializer-nodejs-migration

Conversation

@miguelvelezsa
Copy link
Collaborator

alexander-fenster and others added 30 commits August 2, 2021 15:16
* chore: repository setup

* chore: update auto-approve.yml
Release-As: v0.1.0

* feat!: proto3 JSON serializer and deserializer

* chore: add prelint step

* chore: region tags

* fix: script to run mocha

* chore: no source-map-support/register needed
🤖 I have created a release \*beep\* \*boop\*
---
## 0.1.0 (2021-08-03)


### ⚠ BREAKING CHANGES

* proto3 JSON serializer and deserializer (#2)

### Features

* proto3 JSON serializer and deserializer ([#2](https://www.github.com/googleapis/proto3-json-serializer-nodejs/issues/2)) ([96255a7](https://www.github.com/googleapis/proto3-json-serializer-nodejs/commit/96255a77c7714f33cae547db9160615d7f80a233))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
## Description
Ran into a couple issues with conflicting types / packages because protobufjs was not being imported locally. 

This meant it was using the global `protobuf` namespace before, when it was referencing [`protobuf.Message`](https://github.com/googleapis/proto3-json-serializer-nodejs/blob/main/typescript/src/toproto3json.ts#L44) in the function signature of `toProto3JSON`.



## Checklist

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [ ] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/proto3-json-serializer-nodejs/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [ ] Ensure the tests and linter pass
- [x] Code coverage does not decrease (if any source code was changed)
- [x] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕
If `fromProto3JSON` sees an enum with an unknown value, do not fail. `fromObject` will then likely ignore the unknown value, resulting in an effectively unset enum field, which is still better than throwing an exception.
[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [@types/node](https://togithub.com/DefinitelyTyped/DefinitelyTyped) | [`^14.11.2` -> `^16.0.0`](https://renovatebot.com/diffs/npm/@types%2fnode/14.17.32/16.11.6) | [![age](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/compatibility-slim/14.17.32)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/confidence-slim/14.17.32)](https://docs.renovatebot.com/merge-confidence/) |

---

### Configuration

📅 **Schedule**: "after 9am and before 3pm" (UTC).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/proto3-json-serializer-nodejs).
The fractional seconds in the JSON timestamps can go up to 9 digits (i.e. up to 1 nanosecond resolution). [timestamps.proto](https://github.com/protocolbuffers/protobuf/blob/master/src/google/protobuf/timestamp.proto#L115)
However, Javascript Date object represent any date and time to millisecond precision.

To keep the precision, we extract the fractional seconds and append 0 until the length is equal to 9.
🤖 I have created a release *beep* *boop*
---


### [0.1.7](googleapis/proto3-json-serializer-nodejs@v0.1.6...v0.1.7) (2022-01-14)


### Bug Fixes

* keep nano second precision when maps between JSON and proto3 ([#28](googleapis/proto3-json-serializer-nodejs#28)) ([eaa01ce](googleapis/proto3-json-serializer-nodejs@eaa01ce))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
A timestamp might have no millisecond, form like `2022-01-01T00:00:00Z` vs `2022-01-01T00:00:00.000Z`.
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
* build: enable OwlBot

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
@miguelvelezsa miguelvelezsa requested a review from a team as a code owner October 9, 2025 23:29
@google-cla
Copy link

google-cla bot commented Oct 9, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@miguelvelezsa miguelvelezsa force-pushed the proto3-json-serializer-nodejs-migration branch from 1a0e93d to 529f047 Compare October 9, 2025 23:40
@miguelvelezsa miguelvelezsa changed the title migrate code from googleapis/proto3-json-serializer-nodejs chore: migrate code from googleapis/proto3-json-serializer-nodejs Oct 9, 2025
@miguelvelezsa miguelvelezsa force-pushed the proto3-json-serializer-nodejs-migration branch from 8aa2fae to d3c3748 Compare October 9, 2025 23:44
@miguelvelezsa miguelvelezsa force-pushed the proto3-json-serializer-nodejs-migration branch from 01534ea to f4e9b4b Compare October 9, 2025 23:55
@miguelvelezsa miguelvelezsa merged commit 4bc5e21 into main Oct 10, 2025
12 checks passed
@miguelvelezsa miguelvelezsa deleted the proto3-json-serializer-nodejs-migration branch October 10, 2025 00:08
@miguelvelezsa miguelvelezsa added the release-please:force-run To run release-please label Oct 10, 2025
@release-please release-please bot removed the release-please:force-run To run release-please label Oct 10, 2025
miguelvelezsa pushed a commit that referenced this pull request Oct 10, 2025
GautamSharda pushed a commit that referenced this pull request Oct 29, 2025
GautamSharda pushed a commit that referenced this pull request Oct 29, 2025
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.