@agoric/access-token 0.4.22-other-dev-3eb1a1d.0 → 0.4.22-other-dev-d15096d.0.d15096d

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agoric/access-token",
-  "version": "0.4.22-other-dev-3eb1a1d.0+3eb1a1d",
+  "version": "0.4.22-other-dev-d15096d.0.d15096d",
   "description": "Persistent credentials for Agoric users, backed by a simple JSON file",
   "type": "module",
   "main": "src/access-token.js",
@@ -10,14 +10,15 @@
   "scripts": {
     "build": "exit 0",
     "test": "ava",
-    "test:c8": "c8 --all $C8_OPTIONS ava",
+    "test:c8": "c8 --all ${C8_OPTIONS:-} ava",
     "test:xs": "exit 0",
-    "lint": "run-s --continue-on-error lint:*",
+    "lint": "yarn run -T run-s --continue-on-error 'lint:*'",
     "lint-fix": "yarn lint:eslint --fix",
-    "lint:eslint": "eslint .",
-    "lint:types": "tsc"
+    "lint:eslint": "yarn run -T eslint .",
+    "lint:types": "yarn run -T tsc"
   },
   "dependencies": {
+    "@agoric/internal": "0.3.3-other-dev-d15096d.0.d15096d",
     "n-readlines": "^1.0.0",
     "proper-lockfile": "^4.1.2",
     "tmp": "^0.2.1"
@@ -26,7 +27,7 @@
     "@types/n-readlines": "^1.0.3",
     "@types/proper-lockfile": "^4.1.2",
     "ava": "^5.3.0",
-    "c8": "^10.1.2"
+    "c8": "^10.1.3"
   },
   "publishConfig": {
     "access": "public"
@@ -41,7 +42,10 @@
     "timeout": "2m"
   },
   "typeCoverage": {
-    "atLeast": 83.97
+    "atLeast": 83.57
   },
-  "gitHead": "3eb1a1d2d75b2b4a94807cd3bf759bc9fc531f05"
+  "engines": {
+    "node": "^20.9 || ^22.11"
+  },
+  "gitHead": "d15096dc4ff8b96e9b6cd11954c20d3a9efbb393"
 }

package/src/json-store.js

@@ -5,6 +5,10 @@ import process from 'process';
 import lockfile from 'proper-lockfile';
 import Readlines from 'n-readlines';
 
+/**
+ * @import {LockOptions} from 'proper-lockfile';
+ */
+
 // TODO: Update this when we make a breaking change.
 // const DATA_FILE = 'data.jsonlines';
 //
@@ -159,7 +163,7 @@ function makeStorageInMemory() {
  * @param {string} [dirPath]  Path to a directory in which database files may be kept, or
  *   null.
  * @param {boolean} [forceReset]  If true, initialize the database to an empty state
- * @param {null | import('proper-lockfile').LockOptions['retries']} [lockRetries] If null, do not lock the database.
+ * @param {null | LockOptions['retries']} [lockRetries] If null, do not lock the database.
  *
  * @returns {Promise<{
  *   storage: JSONStore, // a storage API object to load and store data

package/test/state.test.js

@@ -1,5 +1,6 @@
 import test from 'ava';
-import { tmpDir } from './tmp.js';
+import tmp from 'tmp';
+import { makeTempDirFactory } from '@agoric/internal/src/tmpDir.js';
 import {
   initJSONStore,
   openJSONStore,
@@ -7,6 +8,8 @@ import {
   isJSONStore,
 } from '../src/json-store.js';
 
+const tmpDir = makeTempDirFactory(tmp);
+
 function testStorage(t, storage) {
   t.falsy(storage.has('missing'));
   t.is(storage.get('missing'), undefined);
@@ -39,7 +42,7 @@ function testStorage(t, storage) {
 }
 
 test('storageInFile', async t => {
-  const [dbDir, cleanup] = await tmpDir('testdb');
+  const [dbDir, cleanup] = tmpDir('testdb');
   t.teardown(cleanup);
   t.is(isJSONStore(dbDir), false);
   const { storage, commit, close } = await initJSONStore(dbDir);

package/test/token.test.js

@@ -1,10 +1,12 @@
 import test from 'ava';
-import { tmpDir } from './tmp.js';
-
+import tmp from 'tmp';
+import { makeTempDirFactory } from '@agoric/internal/src/tmpDir.js';
 import { getAccessToken } from '../src/access-token.js';
 
+const tmpDir = makeTempDirFactory(tmp);
+
 test('access tokens', async t => {
-  const [sharedStateDir, removeCallback] = await tmpDir('access-token-test');
+  const [sharedStateDir, removeCallback] = tmpDir('access-token-test');
   const [a, b, c] = await Promise.all([
     getAccessToken(1234, sharedStateDir),
     getAccessToken(1234, sharedStateDir),

package/test/tmp.js

@@ -1,19 +0,0 @@
-import tmp from 'tmp';
-
-/**
- * @param {string} [prefix]
- * @returns {Promise<[string, () => void]>}
- */
-export const tmpDir = prefix =>
-  new Promise((resolve, reject) => {
-    // We use `unsafeCleanup` because we want to remove the directory even if it
-    // still contains files.
-    const unsafeCleanup = true;
-    tmp.dir({ unsafeCleanup, prefix }, (err, name, removeCallback) => {
-      if (err) {
-        reject(err);
-      } else {
-        resolve([name, removeCallback]);
-      }
-    });
-  });