Bug: Missing escape characters for long, numeric string identifiers

[lylejohnson]

Describe the bug

A RecordId with a long string of numbers as its identifier is not always jsonified properly. In some cases the long numeric string is not properly escaped.

Steps to reproduce

Here's a simple example to demonstrate the problem:

import { jsonify, RecordId } from 'surrealdb';

jsonify(new RecordId('foo', '125813199042576601589342522460260755'));
=> foo:125813199042576601589342522460260755

Expected behaviour

I would expect the result to be foo:⟨125813199042576601589342522460260755⟩, for consistency with how SurrealDB treats them:

-- string identifier consisting of a long string of numbers
CREATE foo:`125813199042576601589342522460260755`;

[
  {
    id: foo:⟨125813199042576601589342522460260755⟩
  }
]

SurrealDB version

2.1.2

JavaScript SDK version

1.1.0

Contact Details

[email protected]

Is there an existing issue for this?

• I have searched the existing issues

Code of Conduct

• I agree to follow this project's Code of Conduct