@isrd-isi-edu/ermrestjs 2.9.0 → 2.9.1

package/js/hatrac.js

@@ -171,8 +171,8 @@ export function Upload(file, otherInfo) {
   if (!this.file) throw new Error('No file provided while creating hatrac file object');
 
   this.storedFilename = file.name; // the name that will be used for content-disposition and filename column
-  // eslint-disable-next-line @typescript-eslint/no-require-imports, no-undef
-  if (ENV_IS_NODE) this.file.buffer = require('fs').readFileSync(file.path);
+  /* In Node (unit tests) the caller provides `file.buffer`; the browser path reads via FileReader.
+     We avoid importing `fs` here so the browser bundle has no Node built-ins to stub. */
 
   this.column = otherInfo.column;
   if (!this.column) throw new Error('No column provided while creating hatrac file object');

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@isrd-isi-edu/ermrestjs",
   "description": "ERMrest client library in JavaScript",
-  "version": "2.9.0",
+  "version": "2.9.1",
   "license": "Apache-2.0",
   "engines": {
     "node": ">= 20.0.0",
@@ -67,7 +67,7 @@
     "spark-md5": "^3.0.0",
     "terser": "^5.44.1",
     "typescript": "~5.9.3",
-    "vite": "^7.3.2",
+    "vite": "^8.0.16",
     "vite-plugin-compression2": "^2.5.3"
   },
   "devDependencies": {

package/src/models/reference/reference.ts

@@ -1688,7 +1688,7 @@ export class Reference {
   /**
    * Returns true if
    *  - ermrest supports tcrs, and
-   *  - table has dynamic acls, and
+   *  - table or any of the columns of the table have dynamic acls, and
    *  - table has RID column, and
    *  - table is not marked non-updatable by annotation
    */
@@ -1698,7 +1698,7 @@ export class Reference {
       this._canUseTCRS =
         this.table.schema.catalog.features[rightKey] === true &&
         // eslint-disable-next-line eqeqeq
-        this.table.rights[_ERMrestACLs.UPDATE] == null &&
+        (this.table.rights[_ERMrestACLs.UPDATE] == null || this.table.columns.all().some((col) => col.rights[_ERMrestACLs.UPDATE] == null)) &&
         this.table.columns.has('RID') &&
         this.canUpdate;
     }

package/vite.config.mts

@@ -24,7 +24,7 @@ export default defineConfig(async (): Promise<UserConfig> => {
   // dev related plugins
   if (isDev) {
     // visualize the bundle size
-    const { visualizer } = await import('rollup-plugin-visualizer');
+    const { visualizer } = await import(/* @vite-ignore */ 'rollup-plugin-visualizer');
     plugins.push(
       visualizer({
         filename: resolve(__dirname, 'dist', 'stats.html'),
@@ -61,14 +61,12 @@ export default defineConfig(async (): Promise<UserConfig> => {
             name: 'ERMrest',
             format: 'umd',
             entryFileNames: 'ermrest.js',
-            inlineDynamicImports: true,
           },
           // the following is the main build that chaise uses:
           {
             name: 'ERMrest',
             format: 'umd',
             entryFileNames: 'ermrest.min.js',
-            inlineDynamicImports: true,
           },
         ],
       },
@@ -84,6 +82,13 @@ export default defineConfig(async (): Promise<UserConfig> => {
     resolve: {
       alias: {
         '@isrd-isi-edu/ermrestjs': resolve(__dirname),
+        /**
+         * markdown-it imports the deprecated Node `punycode` builtin. In a browser
+         * build that gets externalized to a stub and logs a warning. markdown-it only
+         * calls it inside try/catch for hostname normalization, so an empty shim is
+         * safe (ASCII URLs unaffected) and silences the warning.
+         */
+        punycode: resolve(__dirname, 'vendor/punycode-shim.js'),
       },
     },
     plugins,