@kntic/links 0.1.0 → 0.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kntic/links",
-  "version": "0.1.0",
+  "version": "0.3.0",
   "description": "CLI for building and deploying self-hosted link-in-bio pages. No accounts, no tracking — just HTML.",
   "type": "module",
   "license": "MIT",

package/src/cli.js

@@ -13,6 +13,8 @@ import { registerQr } from './commands/qr.js';
 import { registerConfig } from './commands/config-cmd.js';
 import { registerOpen } from './commands/open-cmd.js';
 import { registerStatus } from './commands/status.js';
+import { registerReorder } from './commands/reorder.js';
+import { registerEdit } from './commands/edit.js';
 
 const require = createRequire(import.meta.url);
 const { version } = require('../package.json');
@@ -35,5 +37,7 @@ registerQr(program);
 registerConfig(program);
 registerOpen(program);
 registerStatus(program);
+registerReorder(program);
+registerEdit(program);
 
 program.parse();

package/src/commands/add.js

@@ -5,7 +5,7 @@
  * atomically. Validates the URL and rejects duplicate labels.
  */
 
-import { findConfig, readConfig, writeConfig, validateUrl } from '../config.js';
+import { findConfig, readConfig, writeConfig, validateUrl, validateDate } from '../config.js';
 
 export function registerAdd(program) {
   program
@@ -25,6 +25,24 @@ export function registerAdd(program) {
         return;
       }
 
+      // Validate schedule dates (if provided)
+      if (opts.from) {
+        const fromCheck = validateDate(opts.from);
+        if (!fromCheck.valid) {
+          console.error(`Error: --from: ${fromCheck.error}`);
+          process.exitCode = 1;
+          return;
+        }
+      }
+      if (opts.until) {
+        const untilCheck = validateDate(opts.until);
+        if (!untilCheck.valid) {
+          console.error(`Error: --until: ${untilCheck.error}`);
+          process.exitCode = 1;
+          return;
+        }
+      }
+
       // Find and read config
       let configPath;
       let config;

package/src/commands/edit.js

@@ -0,0 +1,167 @@
+/**
+ * links edit — Edit an existing link's properties in-place.
+ *
+ * Only specified flags are changed; everything else is left untouched.
+ * Supports --no-<flag> to remove optional fields entirely.
+ */
+
+import { findConfig, readConfig, writeConfig, validateUrl, validateDate } from '../config.js';
+
+export function registerEdit(program) {
+  program
+    .command('edit <label>')
+    .description('Edit an existing link')
+    .option('--label <new-label>', 'Rename the link')
+    .option('--url <url>', 'Update the URL')
+    .option('--icon <emoji>', 'Set or update the icon')
+    .option('--no-icon', 'Remove the icon')
+    .option('--description <text>', 'Set or update the description')
+    .option('--no-description', 'Remove the description')
+    .option('--from <date>', 'Set or update scheduled_from (ISO 8601)')
+    .option('--no-from', 'Remove scheduled_from')
+    .option('--until <date>', 'Set or update scheduled_until (ISO 8601)')
+    .option('--no-until', 'Remove scheduled_until')
+    .action(async (label, opts) => {
+      // Detect whether any editing flags were actually provided.
+      // Commander sets --no-icon to icon=false, --icon <v> to icon=<v>,
+      // and leaves icon=undefined when neither is passed. However,
+      // Commander pre-populates boolean defaults for --no-* pairs:
+      // when neither --icon nor --no-icon is passed, opts.icon is true (default).
+      // We need to check rawArgs or compare against defaults.
+      const hasLabel = opts.label !== undefined;
+      const hasUrl = opts.url !== undefined;
+      // For --no-* pairs: Commander sets default to true. Explicitly passed
+      // --icon <val> gives a string; --no-icon gives false; default is true.
+      const hasIcon = typeof opts.icon === 'string' || opts.icon === false;
+      const hasDescription = typeof opts.description === 'string' || opts.description === false;
+      const hasFrom = typeof opts.from === 'string' || opts.from === false;
+      const hasUntil = typeof opts.until === 'string' || opts.until === false;
+
+      const hasAnyOption = hasLabel || hasUrl || hasIcon || hasDescription || hasFrom || hasUntil;
+
+      if (!hasAnyOption) {
+        console.log(
+          'Usage: links edit <label> [--url <url>] [--label <new>] [--icon <emoji>] [--description <text>] [--from <date>] [--until <date>]',
+        );
+        return;
+      }
+
+      // Validate inputs before touching config
+      if (hasUrl) {
+        const urlCheck = validateUrl(opts.url);
+        if (!urlCheck.valid) {
+          console.error(`Error: ${urlCheck.error}`);
+          process.exitCode = 1;
+          return;
+        }
+      }
+      if (typeof opts.from === 'string') {
+        const fromCheck = validateDate(opts.from);
+        if (!fromCheck.valid) {
+          console.error(`Error: --from: ${fromCheck.error}`);
+          process.exitCode = 1;
+          return;
+        }
+      }
+      if (typeof opts.until === 'string') {
+        const untilCheck = validateDate(opts.until);
+        if (!untilCheck.valid) {
+          console.error(`Error: --until: ${untilCheck.error}`);
+          process.exitCode = 1;
+          return;
+        }
+      }
+
+      // Find and read config
+      let configPath;
+      let config;
+      try {
+        configPath = findConfig();
+        config = readConfig(configPath);
+      } catch (err) {
+        console.error(`Error: ${err.message}`);
+        process.exitCode = 1;
+        return;
+      }
+
+      if (!config.links) {
+        config.links = [];
+      }
+
+      // Find link by label (case-insensitive)
+      const existingIndex = config.links.findIndex(
+        (l) => l.label.toLowerCase() === label.toLowerCase(),
+      );
+
+      if (existingIndex === -1) {
+        console.error(`No link with label "${label}" found.`);
+        process.exitCode = 1;
+        return;
+      }
+
+      // Check rename collision
+      if (hasLabel) {
+        const collision = config.links.findIndex(
+          (l, i) => i !== existingIndex && l.label.toLowerCase() === opts.label.toLowerCase(),
+        );
+        if (collision !== -1) {
+          console.error(`A link with label "${opts.label}" already exists.`);
+          process.exitCode = 1;
+          return;
+        }
+      }
+
+      const link = config.links[existingIndex];
+      const oldLabel = link.label;
+      const renamed = hasLabel && opts.label !== oldLabel;
+
+      // Apply changes — only specified flags
+      if (hasLabel) link.label = opts.label;
+      if (hasUrl) link.url = opts.url;
+
+      if (typeof opts.icon === 'string') {
+        link.icon = opts.icon;
+      } else if (opts.icon === false) {
+        delete link.icon;
+      }
+
+      if (typeof opts.description === 'string') {
+        link.description = opts.description;
+      } else if (opts.description === false) {
+        delete link.description;
+      }
+
+      if (typeof opts.from === 'string') {
+        link.scheduled_from = opts.from;
+      } else if (opts.from === false) {
+        delete link.scheduled_from;
+      }
+
+      if (typeof opts.until === 'string') {
+        link.scheduled_until = opts.until;
+      } else if (opts.until === false) {
+        delete link.scheduled_until;
+      }
+
+      // Write back
+      try {
+        writeConfig(configPath, config);
+      } catch (err) {
+        console.error(`Error: could not write links.yaml — ${err.message}`);
+        process.exitCode = 1;
+        return;
+      }
+
+      // Output message
+      if (renamed) {
+        const otherChanges = hasUrl || hasIcon || hasDescription || hasFrom || hasUntil;
+        if (otherChanges) {
+          console.log(`✓ Renamed "${oldLabel}" → "${link.label}" and updated.`);
+        } else {
+          console.log(`✓ Renamed "${oldLabel}" → "${link.label}"`);
+        }
+      } else {
+        console.log(`✓ Updated: ${link.label}`);
+      }
+    });
+}

package/src/commands/reorder.js

@@ -0,0 +1,263 @@
+/**
+ * links reorder — Reorder links in links.yaml.
+ *
+ * Subcommands:
+ *   (no args)           — print current order with 1-based indices
+ *   move <label> <pos>  — move a link to the given 1-based position
+ *   up <label>          — move a link one position up
+ *   down <label>        — move a link one position down
+ *   set <labels...>     — specify the complete new order by labels
+ */
+
+import { findConfig, readConfig, writeConfig } from '../config.js';
+
+/**
+ * Print the current link order as a numbered list.
+ * @param {object[]} links
+ */
+function printOrder(links) {
+  if (links.length === 0) {
+    console.log('No links configured.');
+    return;
+  }
+  links.forEach((link, i) => {
+    console.log(`${i + 1}. ${link.label}`);
+  });
+}
+
+/**
+ * Find a link index by label (case-insensitive).
+ * @param {object[]} links
+ * @param {string} label
+ * @returns {number} index or -1
+ */
+function findByLabel(links, label) {
+  return links.findIndex(
+    (l) => l.label.toLowerCase() === label.toLowerCase(),
+  );
+}
+
+export function registerReorder(program) {
+  const reorderCmd = program
+    .command('reorder')
+    .description('Reorder links in links.yaml')
+    .action(async () => {
+      // No subcommand — print current order
+      let config;
+      try {
+        const configPath = findConfig();
+        config = readConfig(configPath);
+      } catch (err) {
+        console.error(`Error: ${err.message}`);
+        process.exitCode = 1;
+        return;
+      }
+
+      printOrder(config.links || []);
+    });
+
+  // --- reorder move <label> <position> ---
+  reorderCmd
+    .command('move <label> <position>')
+    .description('Move a link to the given 1-based position')
+    .action(async (label, position) => {
+      let configPath, config;
+      try {
+        configPath = findConfig();
+        config = readConfig(configPath);
+      } catch (err) {
+        console.error(`Error: ${err.message}`);
+        process.exitCode = 1;
+        return;
+      }
+
+      const links = config.links || [];
+      const index = findByLabel(links, label);
+      if (index === -1) {
+        console.error(`No link with label "${label}" found.`);
+        process.exitCode = 1;
+        return;
+      }
+
+      const pos = parseInt(position, 10);
+      if (Number.isNaN(pos) || pos < 1 || pos > links.length) {
+        console.error(`Invalid position: ${position}. Must be between 1 and ${links.length}.`);
+        process.exitCode = 1;
+        return;
+      }
+
+      // Splice out and insert at new position
+      const [item] = links.splice(index, 1);
+      links.splice(pos - 1, 0, item);
+      config.links = links;
+
+      try {
+        writeConfig(configPath, config);
+      } catch (err) {
+        console.error(`Error: could not write links.yaml — ${err.message}`);
+        process.exitCode = 1;
+        return;
+      }
+
+      console.log(`✓ Moved "${item.label}" to position ${pos}.`);
+      printOrder(links);
+    });
+
+  // --- reorder up <label> ---
+  reorderCmd
+    .command('up <label>')
+    .description('Move a link one position up')
+    .action(async (label) => {
+      let configPath, config;
+      try {
+        configPath = findConfig();
+        config = readConfig(configPath);
+      } catch (err) {
+        console.error(`Error: ${err.message}`);
+        process.exitCode = 1;
+        return;
+      }
+
+      const links = config.links || [];
+      const index = findByLabel(links, label);
+      if (index === -1) {
+        console.error(`No link with label "${label}" found.`);
+        process.exitCode = 1;
+        return;
+      }
+
+      if (index === 0) {
+        console.log(`${links[index].label} is already at the top.`);
+        return;
+      }
+
+      // Swap with previous
+      [links[index - 1], links[index]] = [links[index], links[index - 1]];
+      config.links = links;
+
+      try {
+        writeConfig(configPath, config);
+      } catch (err) {
+        console.error(`Error: could not write links.yaml — ${err.message}`);
+        process.exitCode = 1;
+        return;
+      }
+
+      console.log(`✓ Moved "${links[index - 1].label}" up to position ${index}.`);
+      printOrder(links);
+    });
+
+  // --- reorder down <label> ---
+  reorderCmd
+    .command('down <label>')
+    .description('Move a link one position down')
+    .action(async (label) => {
+      let configPath, config;
+      try {
+        configPath = findConfig();
+        config = readConfig(configPath);
+      } catch (err) {
+        console.error(`Error: ${err.message}`);
+        process.exitCode = 1;
+        return;
+      }
+
+      const links = config.links || [];
+      const index = findByLabel(links, label);
+      if (index === -1) {
+        console.error(`No link with label "${label}" found.`);
+        process.exitCode = 1;
+        return;
+      }
+
+      if (index === links.length - 1) {
+        console.log(`${links[index].label} is already at the bottom.`);
+        return;
+      }
+
+      // Swap with next
+      [links[index], links[index + 1]] = [links[index + 1], links[index]];
+      config.links = links;
+
+      try {
+        writeConfig(configPath, config);
+      } catch (err) {
+        console.error(`Error: could not write links.yaml — ${err.message}`);
+        process.exitCode = 1;
+        return;
+      }
+
+      console.log(`✓ Moved "${links[index + 1].label}" down to position ${index + 2}.`);
+      printOrder(links);
+    });
+
+  // --- reorder set <labels...> ---
+  reorderCmd
+    .command('set <labels...>')
+    .description('Specify the complete new order by passing all labels')
+    .action(async (labels) => {
+      let configPath, config;
+      try {
+        configPath = findConfig();
+        config = readConfig(configPath);
+      } catch (err) {
+        console.error(`Error: ${err.message}`);
+        process.exitCode = 1;
+        return;
+      }
+
+      const links = config.links || [];
+
+      if (labels.length !== links.length) {
+        console.error(
+          `Expected ${links.length} labels but got ${labels.length}. All existing labels must be specified.`,
+        );
+        process.exitCode = 1;
+        return;
+      }
+
+      // Validate all provided labels exist and build new order
+      const newLinks = [];
+      const used = new Set();
+
+      for (const label of labels) {
+        const index = findByLabel(links, label);
+        if (index === -1) {
+          console.error(`No link with label "${label}" found.`);
+          process.exitCode = 1;
+          return;
+        }
+
+        const key = links[index].label.toLowerCase();
+        if (used.has(key)) {
+          console.error(`Duplicate label "${label}" in arguments.`);
+          process.exitCode = 1;
+          return;
+        }
+        used.add(key);
+        newLinks.push(links[index]);
+      }
+
+      // Verify all existing labels are covered
+      for (const link of links) {
+        if (!used.has(link.label.toLowerCase())) {
+          console.error(`Missing label "${link.label}" — all existing labels must be specified.`);
+          process.exitCode = 1;
+          return;
+        }
+      }
+
+      config.links = newLinks;
+
+      try {
+        writeConfig(configPath, config);
+      } catch (err) {
+        console.error(`Error: could not write links.yaml — ${err.message}`);
+        process.exitCode = 1;
+        return;
+      }
+
+      console.log('✓ Links reordered.');
+      printOrder(newLinks);
+    });
+}

package/src/config.js

@@ -1,20 +1,7 @@
-import { readFileSync, writeFileSync, renameSync } from 'node:fs';
+import { readFileSync, writeFileSync, renameSync, existsSync } from 'node:fs';
 import { resolve, dirname, join } from 'node:path';
 import yaml from 'js-yaml';
 
-// ---------------------------------------------------------------------------
-// Schema defaults
-// ---------------------------------------------------------------------------
-
-export const DEFAULT_CONFIG = {
-  name: 'My Links',
-  bio: '',
-  avatar: '',
-  theme: 'minimal-dark',
-  domain: '',
-  links: [],
-};
-
 // ---------------------------------------------------------------------------
 // Validation helpers
 // ---------------------------------------------------------------------------
@@ -154,7 +141,12 @@ export function findConfig(startDir = process.cwd()) {
       readFileSync(candidate); // existence check
       return candidate;
     } catch {
-      // not found — go up
+      // not found — check for git root boundary before climbing
+      if (existsSync(join(dir, '.git'))) {
+        throw new Error(
+          'links.yaml not found (stopped at git root). Run "links init" to create a new project.',
+        );
+      }
     }
 
     const parent = dirname(dir);

package/src/generator.js

@@ -9,6 +9,7 @@ import { readFileSync } from 'node:fs';
 import { resolve, dirname, extname } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { isLinkActive } from './utils.js';
+import { loadTheme } from './themes/loader.js';
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 
@@ -27,6 +28,15 @@ function esc(str) {
     .replace(/'/g, ''');
 }
 
+/**
+ * Extract the --accent-color value from a CSS string.
+ * Falls back to '#6366f1' (indigo) if not found.
+ */
+export function extractAccentColor(css) {
+  const match = css.match(/--accent-color:\s*([^;]+);/);
+  return match ? match[1].trim() : '#6366f1';
+}
+
 /** MIME type lookup for common image formats. */
 function imageMime(filePath) {
   const ext = extname(filePath).toLowerCase();
@@ -70,27 +80,6 @@ export function filterScheduled(links, now = new Date()) {
   return links.filter((link) => isLinkActive(link, now));
 }
 
-// ---------------------------------------------------------------------------
-// Theme CSS loading
-// ---------------------------------------------------------------------------
-
-/**
- * Load theme CSS from the themes directory.
- * @param {string} themeName — name without .css extension
- * @returns {string} CSS content
- */
-export function loadThemeCSS(themeName) {
-  const themePath = resolve(__dirname, 'themes', `${themeName}.css`);
-  try {
-    return readFileSync(themePath, 'utf8');
-  } catch {
-    throw new Error(
-      `Theme "${themeName}" not found at ${themePath}. ` +
-      'Available themes live in src/themes/.',
-    );
-  }
-}
-
 // ---------------------------------------------------------------------------
 // HTML generation
 // ---------------------------------------------------------------------------
@@ -113,7 +102,7 @@ export function generatePage(config, options = {}) {
   const theme = config.theme || 'minimal-dark';
 
   // Load and inline CSS
-  const css = loadThemeCSS(theme);
+  const css = loadTheme(theme);
 
   // Filter links by schedule
   const allLinks = config.links || [];
@@ -124,16 +113,37 @@ export function generatePage(config, options = {}) {
 
   // Avatar handling
   let avatarHTML = '';
+  let avatarDataUri = null;
   if (config.avatar && config.avatar.trim().length > 0) {
     try {
-      const dataUri = inlineImage(config.avatar, configDir);
-      avatarHTML = `<img class="profile__avatar" src="${dataUri}" alt="${esc(name)}" width="88" height="88">`;
+      avatarDataUri = inlineImage(config.avatar, configDir);
+      avatarHTML = `<img class="profile__avatar" src="${avatarDataUri}" alt="${esc(name)}" width="88" height="88">`;
     } catch {
       // Fallback: reference the file directly (it will be copied to output dir)
       avatarHTML = `<img class="profile__avatar" src="${esc(config.avatar)}" alt="${esc(name)}" width="88" height="88">`;
     }
   }
 
+  // Favicon handling — Strategy A (avatar) or Strategy B (SVG letter)
+  let faviconHTML = '';
+  if (avatarDataUri) {
+    // Strategy A: reuse the already-computed avatar data URI
+    const avatarMime = avatarDataUri.startsWith('data:image/svg') ? 'image/svg+xml' : 'image/png';
+    faviconHTML = `  <link rel="icon" type="${avatarMime}" href="${avatarDataUri}">\n`
+               + `  <link rel="icon" type="${avatarMime}" sizes="32x32" href="${avatarDataUri}">`;
+  } else {
+    // Strategy B: generate SVG favicon from first letter of name
+    const firstChar = name[0].toUpperCase();
+    const accent = extractAccentColor(css);
+    const svg = `<svg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 64 64'>`
+              + `<rect width='64' height='64' rx='12' fill='${accent}'/>`
+              + `<text x='32' y='32' text-anchor='middle' dominant-baseline='central' font-family='system-ui,sans-serif' font-size='32' font-weight='700' fill='white'>${firstChar}</text>`
+              + `</svg>`;
+    const svgDataUri = `data:image/svg+xml;base64,${Buffer.from(svg).toString('base64')}`;
+    faviconHTML = `  <link rel="icon" type="image/svg+xml" href="${svgDataUri}">\n`
+               + `  <link rel="icon" type="image/svg+xml" sizes="32x32" href="${svgDataUri}">`;
+  }
+
   // Build link list HTML
   const linksHTML = links.map((link) => {
     const icon = link.icon ? `<span class="links__icon">${esc(link.icon)}</span>` : '';
@@ -162,6 +172,7 @@ export function generatePage(config, options = {}) {
   <meta property="og:title" content="${esc(name)}">
   <meta property="og:description" content="${esc(ogDescription)}">
   <meta property="og:type" content="website">
+${faviconHTML}
   <style>
 ${css}
   </style>

package/src/utils.js

@@ -1,27 +1,11 @@
 /**
  * Shared utility helpers for @kntic/links.
+ *
+ * Exports:
+ *   - fatal(message, code) — pretty-print error and exit
+ *   - isLinkActive(link, now) — schedule-aware link visibility check
  */
 
-import { readFile } from 'node:fs/promises';
-import { resolve } from 'node:path';
-import yaml from 'js-yaml';
-
-/**
- * Default config filename.
- */
-export const CONFIG_FILE = 'links.yml';
-
-/**
- * Load and parse the links YAML config from the current directory.
- * @param {string} [dir=process.cwd()] - Directory to look in.
- * @returns {Promise<object>} Parsed config object.
- */
-export async function loadConfig(dir = process.cwd()) {
-  const filePath = resolve(dir, CONFIG_FILE);
-  const raw = await readFile(filePath, 'utf8');
-  return yaml.load(raw);
-}
-
 /**
  * Pretty-print an error and exit.
  * @param {string} message