@venizia/vulcan 0.1.0-1 → 0.1.0-2

package/base/rules.md

@@ -58,7 +58,7 @@ classes) and `V-C-05` (`getError` / `ApplicationError`) - the last two, with the
 | ID | Rule | Enforced by |
 |----|------|-------------|
 | V-C-01 | **English only in code** - identifiers, comments, commit messages, logs, docs. Vietnamese belongs in chat. | SOFT |
-| V-C-02 | **Options objects everywhere:** `fn({ key, value })`, never `fn(key, value)`. **One parameter is not an exception** - `isValidName({ name })`, not `isValidName(name)`. A single positional parameter reads fine until the second one arrives, and by then the method is published and every call site pays. The cost is highest on an interface others implement: `IStorageHelper` carried three positional methods for exactly this reason and the migration touched 72 call sites. Mixing the two styles in one interface is itself the defect - a reader cannot tell which shape a method takes without opening it. | SOFT |
+| V-C-02 | **One options object first, at most 5 parameters.** Fold every argument you can into a single options object in the first position: `fn({ key, value })`, not `fn(key, value)`, and one parameter is not an exception - `isValidName({ name })`, not `isValidName(name)`. A second parameter is allowed only where the first object cannot carry it, for example a repository method that takes its data and its query options separately. Never more than 5 parameters. A single positional parameter reads fine until the second one arrives, and by then the method is published and every call site pays. The cost is highest on an interface others implement: `IStorageHelper` carried three positional methods for exactly this reason and the migration touched 72 call sites. Mixing the two styles in one interface is itself the defect - a reader cannot tell which shape a method takes without opening it. | SOFT - lint (`venizia/max-params-own`, 5) once `@venizia/dev-configs` ships `houseConfigs` |
 | V-C-04 | **Never abbreviate identifiers:** `ProductRepository` not `ProductRepo`, `ProductDocument` not `ProductDoc`; type parameters too (`TDocument`, not `TDoc`). A project lists its own exceptions in its rules (IGNIS: `tz` for a time zone). | SOFT |
 | V-C-06 | **Never a silent catch - log it.** Always braces. Early return. `switch` with `default` over a long if-else chain. Strict TypeScript; no `any` where a type exists. | LINT (braces, floating promises) + SOFT |
 | V-C-07 | **A function name opens with its verb** - `generate` `build` `to` `is` `has` `assert` `extract` `enrich` `get` `resolve`. `is*` returns a boolean and leaves the branch to the caller; `assert*` throws and returns `void`, so the code after it needs none - reach for `assert*` when every caller would throw on `false` anyway. `has*` is the ownership question (`hasBucket`, never `isBucketExists`). That list is for UTILITY functions; service methods lean on a wider set. **Arrow functions, not `function` declarations** (do not churn existing ones). State plus the operations on it live in a class with static members - the `LoggerFactory` shape - never module-level `let` plus exported arrows. | SOFT |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@venizia/vulcan",
-  "version": "0.1.0-1",
+  "version": "0.1.0-2",
   "description": "VULCAN - one setup, one workflow for every project: init, sync, doctor, upgrade, and `vulcan x` for the shipped tools. Bun-only.",
   "homepage": "https://github.com/VENIZIA-AI/vulcan/tree/main/packages/cli",
   "repository": {
@@ -43,9 +43,8 @@
     "test": "NODE_ENV=test bun test --env-file=.env.test"
   },
   "dependencies": {
-    "@venizia/ignis-helpers": "^0.2.2-5",
-    "@venizia/vulcan-atlas": "0.1.0-1",
-    "@venizia/vulcan-okf": "0.1.0-1",
+    "@venizia/vulcan-atlas": "0.1.0-2",
+    "@venizia/vulcan-okf": "0.1.0-2",
     "zod": "^4.5.4"
   },
   "devDependencies": {

package/src/common/types.ts

@@ -1,6 +1,8 @@
-import type { TConstValue } from '@venizia/ignis-helpers/common';
 import type { VulcanCommands, VulcanTools } from './constants.ts';
 
+/** The values of a const class's static fields: `TConstValue<typeof Colors>` is `'red' | 'blue'`. */
+export type TConstValue<TClass> = Extract<TClass[keyof TClass], string | number> & {};
+
 export type TVulcanCommand = TConstValue<typeof VulcanCommands>;
 export type TVulcanTool = TConstValue<typeof VulcanTools>;
 

package/src/config/common/types.ts

@@ -1,4 +1,4 @@
-import type { TConstValue } from '@venizia/ignis-helpers/common';
+import type { TConstValue } from '../../common/index.ts';
 import type { CodingAgents, ConfigExtensions, HookEvents } from './constants.ts';
 
 export type TConfigExtension = TConstValue<typeof ConfigExtensions>;

package/src/doctor/common/types.ts

@@ -1,4 +1,4 @@
-import type { TConstValue } from '@venizia/ignis-helpers/common';
+import type { TConstValue } from '../../common/index.ts';
 import type { DoctorChecks } from './constants.ts';
 
 export type TDoctorCheck = TConstValue<typeof DoctorChecks>;

package/src/hooks/common/types.ts

@@ -1,4 +1,4 @@
-import type { TConstValue } from '@venizia/ignis-helpers/common';
+import type { TConstValue } from '../../common/index.ts';
 import type { GuardDecisions } from './constants.ts';
 
 export type TGuardDecision = TConstValue<typeof GuardDecisions>;

package/src/init/common/types.ts

@@ -1,4 +1,4 @@
-import type { TConstValue } from '@venizia/ignis-helpers/common';
+import type { TConstValue } from '../../common/index.ts';
 import type { InitStatuses } from './constants.ts';
 
 export type TInitStatus = TConstValue<typeof InitStatuses>;

package/src/sync/common/types.ts

@@ -1,4 +1,4 @@
-import type { TConstValue } from '@venizia/ignis-helpers/common';
+import type { TConstValue } from '../../common/index.ts';
 import type { SyncStatuses } from './constants.ts';
 
 export type TSyncStatus = TConstValue<typeof SyncStatuses>;