@joystick.js/node-canary 0.0.0-canary.356 → 0.0.0-canary.357

package/dist/app/accounts/roles/index.js

@@ -1 +1 @@
-import o from"./add.js";import m from"./remove.js";import e from"./list.js";import t from"./grant.js";import f from"./revoke.js";import r from"./user_has_role.js";var u={add:o,remove:m,list:e,grant:t,revoke:f,userHasRole:r,user_has_role:r};export{u as default};
+import o from"./add.js";import m from"./remove.js";import e from"./list.js";import t from"./grant.js";import f from"./revoke.js";import r from"./user_has_role.js";const i={add:o,remove:m,list:e,grant:t,revoke:f,userHasRole:r,user_has_role:r};var n=i;export{n as default};

package/increment_version.js

@@ -1,14 +1,3 @@
 import child_process from 'child_process';
 
-// import fs from 'fs';
-
-// const raw_package_json = fs.readFileSync('package.json', 'utf-8');
-// const package_json = JSON.parse(raw_package_json);
-// const version_parts = package_json.version.split('.');
-// const current_patch_version = version_parts?.pop();
-
-// package_json.version = `${version_parts?.join('.')}.${parseInt(current_patch_version, 10) + 1}`;
-
-// fs.writeFileSync('package.json', JSON.stringify(package_json, null, 2));
-
 child_process.execSync(`npm version prerelease --preid=canary`);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@joystick.js/node-canary",
   "type": "module",
-  "version": "0.0.0-canary.356",
+  "version": "0.0.0-canary.357",
   "description": "The Node.js framework for Joystick.",
   "main": "./dist/index.js",
   "scripts": {

package/src/app/accounts/roles/index.js

@@ -5,13 +5,14 @@ import grant from './grant.js';
 import revoke from './revoke.js';
 import user_has_role from './user_has_role.js';
 
-export default {
+const roles = {
   add,
   remove,
   list,
   grant,
   revoke,
-  // NOTE: Keep camelCase here for backwards compatibility.
   userHasRole: user_has_role,
   user_has_role,
-};
+};
+
+export default roles;

package/src/app/api/handle_api_error.js

@@ -2,13 +2,10 @@ import format_api_error from "./format_api_error.js";
 import types from "../../lib/types.js";
 
 const handle_api_error = (location = '', error = '', res = {}) => {
-	// NOTE: This is an internal Joystick error that we control.
+	// NOTE: This is an internal Joystick error that we control. Should be
+	// in the shape of { errors: [] } pre-formatted.
 	if (types.is_object(error) && !(error instanceof Error)) {
     return res.status((error?.errors && error?.errors[0]?.status) || 500).send(
-    	// TODO: This should match the output of format_api_error(). We can either
-    	// do a mapping here, or, ensure that when we *do* return an object with
-    	// an errors array, each error in that array is shaped identical to the
-    	// output of format_api_error().
       JSON.stringify(error)
     );
 	}