mv ./lib/session.js ./lib/session/index.js

Author: msimerson

.github/copilot-instructions.md

@@ -23,7 +23,7 @@ Local tests expect MySQL plus the NicTool schema. CI initializes it with `sh sql
 - `lib/<resource>/index.js` is usually a backend selector. The active implementation comes from `NICTOOL_DATA_STORE` and defaults to MySQL; some resources also support TOML, MongoDB, or Elasticsearch backends.
 - The real persistence logic lives under `lib/<resource>/store/*.js`. These stores translate between API-friendly field names and the legacy NicTool schema (`nt_*` columns), using helpers like `mapToDbColumn`, `objectToDb`, and `dbToObject`.
 - `lib/mysql.js` is the shared query builder/executor. Repositories rely on it for SQL generation rather than embedding ad hoc parameter handling in route files.
-- Auth is session-backed but bearer-token based: `POST /session` authenticates through the user repo, creates an `nt_user_session` row via `lib/user/session.js`, and returns a JWT. Almost every other route runs under the default JWT auth strategy.
+- Auth is session-backed but bearer-token based: `POST /session` authenticates through the session repo, creates an `nt_user_session` row via `lib/session/index.js`, and returns a JWT. Almost every other route runs under the default JWT auth strategy.
 - Group, user, and permission behavior is coupled. Group creation also creates a group-level permission row, user reads attach effective permissions, and permission resolution falls back from explicit user permissions to group permissions.
 - DNS data has an extra translation layer. `lib/zone_record/store/mysql.js` validates records with `@nictool/dns-resource-record` and maps NicTool’s legacy zone-record columns to RFC-style record fields.
 - Config comes from `conf.d/*.toml`, with runtime overrides from `NICTOOL_DB_*` and `NICTOOL_HTTP_*`. `lib/config.js` also auto-loads TLS material from any `.pem` file in `conf.d/`.

lib/user/session.js lib/session/index.jslib/user/session.js renamed to lib/session/index.js

@@ -1,9 +1,9 @@
 import assert from 'node:assert/strict'
 import { describe, it, after, before } from 'node:test'
 
-import User from '../index.js'
-import Session from '../session.js'
-import userCase from './user.json' with { type: 'json' }
+import User from '../../user/index.js'
+import Session from '../index.js'
+import userCase from '../../user/test/user.json' with { type: 'json' }
 
 const sessionUser = {
   ...userCase,

lib/user/test/session.js lib/session/test/index.jslib/user/test/session.js renamed to lib/session/test/index.js

@@ -4,7 +4,7 @@ import Config from '../lib/config.js'
 import Jwt from '@hapi/jwt'
 
 import User from '../lib/user/index.js'
-import Session from '../lib/user/session.js'
+import Session from '../lib/session/index.js'
 
 import { meta } from '../lib/util.js'
 

routes/session.js

@@ -4,7 +4,7 @@ import path from 'node:path'
 
 import Group from './lib/group/index.js'
 import User from './lib/user/index.js'
-import Session from './lib/user/session.js'
+import Session from './lib/session/index.js'
 import Permission from './lib/permission/index.js'
 import Nameserver from './lib/nameserver/index.js'
 import Zone from './lib/zone/index.js'

test-fixtures.js

@@ -30,5 +30,5 @@ else
 		# npm i --no-save node-test-github-reporter
 		# $NODE --test --test-reporter=node-test-github-reporter
 	# fi
-	$NODE --test --test-reporter=spec lib/*/test/*.js lib/*.test.js routes/*.test.js
+	$NODE --test --test-reporter=spec lib/*/test/index.js lib/*.test.js routes/*.test.js
 fi