Repository navigation
Make jetdb-wasm a package npm can show: its README and the JavaScript API - #118
Merged
Merged
Conversation
- Tell what the package is, how to install it, how to use it in a browser and in Node.js, what it reads with which methods, how values and errors come, and its limitations - Point to docs/wasm.md for each method and its options Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…s page - build-wasm-package.sh copies README.md into the package with the licenses - test-wasm-package.sh checks that the package npm installs has README.md - Add homepage, the README of jetdb-wasm, and bugs, the issues on GitHub, to package.json Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- Write docs/wasm.md: how to load the module, open a database and free it, the methods and options for tables, relationships and DDL, queries, VBA, forms and reports, macros, and object properties, the values, the kinds of errors, and the limitations - Leave the fields of each object to the comments of the TypeScript types in the package Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- Write docs/wasm.ja.md in the same sections as docs/wasm.md: loading the module, opening and freeing a database, each method and its options, the values, the kinds of errors, and the limitations - Name the Access types Large Number and Attachment as Microsoft's Japanese article does Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…orts and VBA of Access 97 - Add to the limitations that jetdb does not read the forms, reports and VBA of an Access 97 (Jet3) file, as docs/wasm.md tells Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- Use Yes/No型, 拡張した日付/時刻 and レプリケーション ID型 (GUID) in the Japanese version, as the lists of data types and field sizes in the table design view of Access name them - Use Short Text, Long Text and Replication ID (GUID) in the English version, as Microsoft's English article names them Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- A bundler that builds for the server, such as for server-side rendering, gets the CommonJS build through the node condition of the exports, and that build has no default export to call as init - Tell in docs/wasm.md, wasm.ja.md and the README which condition chooses which build, and that init() is called only with the ES module Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ID_ARGUMENT without type - properties, when objects of different types share the name, and formControls, formProperties, formStream and embeddedMacros, when a form and a report share it, throw INVALID_ARGUMENT without the type option, while the documentation told only of an argument of the wrong kind - Tell it in the table of errors and with the type options (en/ja) Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…t API - Add, as the TypeScript types tell, that a named macro whose grid of actions could not be read and an embedded macro stored without XML have no text either, besides a data macro (en/ja) Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…of jetdb-wasm - Tell that a Date/Time Extended value that cannot be read as a date and time comes as a Uint8Array of its bytes, a type jetdb does not know as a Uint8Array, and a complex column of a kind jetdb does not know as a number, the ID of its values in a hidden table (en/ja) Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
… import - JetdbError is only a TypeScript type, and the package exports no class of the name, so importing it to tell errors by instanceof fails, on import or at the check - Tell in the README, docs/wasm.md and wasm.ja.md that the methods throw an Error whose name is JetdbError, to be told by name and code, as the example now does Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- Add the check of the README in test-wasm-package.sh to the description of the package test in docs/ci.md and ci.ja.md Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- The check of the README in test-wasm-package.sh stopped without a word under set -e; it now says that the package has no README.md before it stops Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ipt API - The README of the repository leads to the documentation of the CLI and the library, but only the README for npm linked to docs/wasm.md - Add to the features that the jetdb-wasm package in crates/jetdb-wasm reads databases in the browser and Node.js, with a link to docs/wasm.md (docs/wasm.ja.md in Japanese); it is not on npm yet, so no installation is told Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- next is a position among the data pages of the table, and points to another row in another table of the same file, while the TypeScript types and the documentation called it a position in the file, valid for the same file - Call it a position among the pages of the table, valid only for the same table of the same file, in the comment of the types (js.rs), docs/wasm.md and wasm.ja.md Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ation - INVALID_ARGUMENT is thrown also for a value of the right type that is not valid, such as a limit that is not a positive integer, a delimiter that is not one character, a bin outside its list, or a start that is not an earlier next; the documentation told only of an argument of the wrong kind and of a shared name, and the comment of the types only of the former - Tell both an argument or option that is not valid and a name shared by objects without the type option, in the comment of the types (js.rs), docs/wasm.md and wasm.ja.md Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- Tell that using works only where JavaScript supports it, as the generated code defines Symbol.dispose only where it exists - Say in the Japanese version, as the English one does, that every property has its number as id - Tell that the xml of a data macro is the whole XML document it was stored in, which may hold other data macros of the table - Add to the table of values that Date/Time is rounded to the second and that multiple values give an empty array when there are none (en/ja) Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
… it is chosen - The example read the file of the input as the page loaded, so copied as it was, it failed on a file not chosen yet - Open the chosen file on the change event of the input Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- Reword the values that come as strings in the README, and in the English version the sentence on server-side rendering, the limit option, the names of properties, and the cases in which text is null - Call it サーバーサイドレンダリング (SSR) and 取得 in the Japanese version, and reword the queries and the names of the columns - Add, as the comment of the types does, that Math.fround(value) gives the stored value of a Single (en/ja) Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
🤖 Generated with Claude Code