codex-configurator 0.2.0 → 0.2.2

package/README.md

@@ -47,6 +47,7 @@ The right-hand pane shows what each setting means, plus a picker when a value ha
 Deprecated settings are marked with a `[!]` warning marker; only that marker is highlighted.
 Model picker entries are curated presets maintained by this project.
 In select lists, `[default]` marks the default option.
+The header banner shows the configurator package version (for example `v0.2.1`) next to the wordmark.
 
 ## TOML-aware navigation
 
@@ -60,6 +61,7 @@ The table view follows TOML structure, with a root catalog of common keys:
 - Attributes and subattributes are shown in strict alphabetical order.
 - Unset boolean settings display explicit defaults as `true [default]` or `false [default]`.
 - For placeholder keys like `<path>`, IDs entered in the UI are normalized under your home directory, and traversal outside home is rejected.
+- New `<path>` entries are written as explicit tables (for example `[projects."/home/me/repo"]`) instead of inline empty objects.
 
 - Dotted/table sections become navigable table nodes.
 - Inline key-value pairs are shown as leaf entries.
@@ -95,12 +97,12 @@ When the log exceeds the size limit, it is rotated to `<log-path>.1` before writ
 
 ## Version checks
 
-Codex version checks are disabled by default to avoid executing unknown binaries from `PATH`.
-To enable checks, all of the following must be set:
+Codex version checks are always enabled.
+By default, the app runs global `codex` and `npm` from `PATH`.
+You can override either command with:
 
-- `CODEX_CONFIGURATOR_ENABLE_VERSION_CHECK=1`
-- `CODEX_CONFIGURATOR_CODEX_BIN=/absolute/path/to/codex`
-- `CODEX_CONFIGURATOR_NPM_BIN=/absolute/path/to/npm`
+- `CODEX_CONFIGURATOR_CODEX_BIN=<command-or-path>`
+- `CODEX_CONFIGURATOR_NPM_BIN=<command-or-path>`
 
 ## Upstream reference
 

package/index.js

@@ -2,7 +2,7 @@
 
 import React, { useState, useEffect } from 'react';
 import { execFile } from 'node:child_process';
-import path from 'node:path';
+import { createRequire } from 'node:module';
 import { render, useInput, useApp, useStdout, Text, Box } from 'ink';
 import { CONTROL_HINT, EDIT_CONTROL_HINT } from './src/constants.js';
 import {
@@ -31,6 +31,9 @@ import {
 import { Header } from './src/components/Header.js';
 import { ConfigNavigator } from './src/components/ConfigNavigator.js';
 
+const require = createRequire(import.meta.url);
+const { version: PACKAGE_VERSION = 'unknown' } = require('./package.json');
+
 const computeListViewportHeight = (rows, terminalRows) =>
   Math.max(4, Math.min(rows.length, Math.min(20, Math.max(4, terminalRows - 14))));
 
@@ -58,8 +61,6 @@ const isCustomIdTableRow = (pathSegments, row) =>
 
 const isInlineTextMode = (mode) => mode === 'text' || mode === 'add-id';
 const VERSION_COMMAND_TIMEOUT_MS = 3000;
-const VERSION_DISABLED_LABEL = 'version check disabled';
-const VERSION_CHECK_ENABLED_ENV_VAR = 'CODEX_CONFIGURATOR_ENABLE_VERSION_CHECK';
 const CODEX_BIN_ENV_VAR = 'CODEX_CONFIGURATOR_CODEX_BIN';
 const NPM_BIN_ENV_VAR = 'CODEX_CONFIGURATOR_NPM_BIN';
 
@@ -85,31 +86,15 @@ const runCommand = (command, args = []) =>
     );
   });
 
-const getAbsoluteCommandPath = (environmentVariableName) => {
-  const configuredPath = String(process.env[environmentVariableName] || '').trim();
-  if (!configuredPath || !path.isAbsolute(configuredPath)) {
-    return '';
-  }
-
-  return configuredPath;
+const getConfiguredCommand = (environmentVariableName, fallbackCommand) => {
+  const configuredCommand = String(process.env[environmentVariableName] || '').trim();
+  return configuredCommand || fallbackCommand;
 };
 
-const getVersionCommands = () => {
-  if (process.env[VERSION_CHECK_ENABLED_ENV_VAR] !== '1') {
-    return null;
-  }
-
-  const codexCommand = getAbsoluteCommandPath(CODEX_BIN_ENV_VAR);
-  const npmCommand = getAbsoluteCommandPath(NPM_BIN_ENV_VAR);
-  if (!codexCommand || !npmCommand) {
-    return null;
-  }
-
-  return {
-    codexCommand,
-    npmCommand,
-  };
-};
+const getVersionCommands = () => ({
+  codexCommand: getConfiguredCommand(CODEX_BIN_ENV_VAR, 'codex'),
+  npmCommand: getConfiguredCommand(NPM_BIN_ENV_VAR, 'npm'),
+});
 
 const getCodexVersion = async (codexCommand) => {
   const output = await runCommand(codexCommand, ['--version']);
@@ -157,14 +142,6 @@ const compareVersions = (left, right) => {
 
 const getCodexUpdateStatus = async () => {
   const commands = getVersionCommands();
-  if (!commands) {
-    return {
-      installed: VERSION_DISABLED_LABEL,
-      latest: 'unknown',
-      status: '',
-    };
-  }
-
   const installedLabel = await getCodexVersion(commands.codexCommand);
   const installed = normalizeVersion(installedLabel);
 
@@ -785,7 +762,11 @@ const App = () => {
     return React.createElement(
       Box,
       { flexDirection: 'column', padding: 1 },
-      React.createElement(Header, { codexVersion, codexVersionStatus }),
+      React.createElement(Header, {
+        codexVersion,
+        codexVersionStatus,
+        packageVersion: PACKAGE_VERSION,
+      }),
       React.createElement(ConfigNavigator, {
         snapshot,
         pathSegments,
@@ -803,7 +784,11 @@ const App = () => {
   return React.createElement(
     Box,
     { flexDirection: 'column', padding: 1 },
-    React.createElement(Header, { codexVersion, codexVersionStatus }),
+    React.createElement(Header, {
+      codexVersion,
+      codexVersionStatus,
+      packageVersion: PACKAGE_VERSION,
+    }),
     React.createElement(ConfigNavigator, {
       snapshot,
       pathSegments,

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "codex-configurator",
-	"version": "0.2.0",
+	"version": "0.2.2",
 	"description": "TOML-aware Ink TUI for Codex Configurator",
 	"main": "index.js",
 	"type": "module",

package/src/components/Header.js

@@ -10,7 +10,7 @@ const WORDMARK = [
   ' ╚═════╝ ╚═════╝ ╚═════╝ ╚══════╝╚═╝  ╚═╝     ╚═════╝ ╚═════╝ ╚═╝  ╚═══╝╚═╝     ╚═╝ ╚═════╝  ╚═════╝ ╚═╝  ╚═╝╚═╝  ╚═╝   ╚═╝    ╚═════╝ ╚═╝  ╚═╝',
 ];
 
-export const Header = ({ codexVersion, codexVersionStatus }) =>
+export const Header = ({ codexVersion, codexVersionStatus, packageVersion }) =>
   React.createElement(
     Box,
     {
@@ -20,14 +20,34 @@ export const Header = ({ codexVersion, codexVersionStatus }) =>
     },
       React.createElement(
         Box,
-        { flexDirection: 'column', marginBottom: 1, gap: 0 },
-        ...WORDMARK.map((line, index) =>
-          React.createElement(Text, { color: 'magentaBright', bold: true, key: `word-${index}` }, line)
-        )
-      ),
+        { flexDirection: 'row', marginBottom: 1, gap: 0, alignItems: 'flex-end' },
+        React.createElement(
+          Box,
+          { flexDirection: 'column' },
+          ...WORDMARK.map((line, index) =>
+            React.createElement(Text, { color: 'magentaBright', bold: true, key: `word-${index}` }, line)
+          )
+        ),
+      React.createElement(
+        Box,
+        { marginLeft: 1 },
+        React.createElement(Text, { color: 'gray', bold: true }, `v${packageVersion || 'unknown'}`)
+      )
+    ),
     React.createElement(
-      Text,
-      { color: 'gray' },
-      codexVersionStatus ? `Codex ${codexVersion} (${codexVersionStatus})` : `Codex ${codexVersion}`
+      Box,
+      { flexDirection: 'row' },
+      React.createElement(Text, { color: 'gray' }, `Codex ${codexVersion}`),
+      codexVersionStatus
+        ? codexVersionStatus === 'up to date'
+          ? React.createElement(
+              React.Fragment,
+              null,
+              React.createElement(Text, { color: 'gray' }, ' ('),
+              React.createElement(Text, { color: 'green' }, '✓'),
+              React.createElement(Text, { color: 'gray' }, ` ${codexVersionStatus})`)
+            )
+          : React.createElement(Text, { color: 'gray' }, ` (${codexVersionStatus})`)
+        : null
     )
   );

package/src/configParser.js

@@ -171,6 +171,32 @@ export const readConfig = (configPath = CONFIG_PATH) => {
 };
 
 const normalizeFilePath = (outputPath) => outputPath || CONFIG_PATH;
+const EMPTY_PATH_TABLE_MARKER_KEY = '__codex_configurator_empty_path_table_marker__';
+
+const cloneForTomlWrite = (value, pathSegments = []) => {
+  if (Array.isArray(value)) {
+    return value.map((item, index) => cloneForTomlWrite(item, [...pathSegments, String(index)]));
+  }
+
+  if (!isPlainObject(value)) {
+    return value;
+  }
+
+  const customPlaceholder = getReferenceCustomIdPlaceholder(pathSegments);
+  const clonedEntries = Object.entries(value).map(([key, child]) => {
+    if (
+      customPlaceholder === '<path>' &&
+      isPlainObject(child) &&
+      Object.keys(child).length === 0
+    ) {
+      return [key, { [EMPTY_PATH_TABLE_MARKER_KEY]: undefined }];
+    }
+
+    return [key, cloneForTomlWrite(child, [...pathSegments, key])];
+  });
+
+  return Object.fromEntries(clonedEntries);
+};
 
 export const writeConfig = (data, outputPath = CONFIG_PATH) => {
   const targetPath = normalizeFilePath(outputPath);
@@ -186,7 +212,7 @@ export const writeConfig = (data, outputPath = CONFIG_PATH) => {
       fs.mkdirSync(directoryPath, { recursive: true, mode: 0o700 });
     }
 
-    const payload = stringify(data);
+    const payload = stringify(cloneForTomlWrite(data));
     let tempFd = null;
 
     try {