reslop 0.1.8 → 0.2.0

package/lib/select.js

@@ -145,8 +145,6 @@ const parseCells = (line) => {
 
 const rgbKey = (rgb) => (rgb ? `${rgb[0]},${rgb[1]},${rgb[2]}` : '');
 
-const styleKey = (fg, bg) => `${rgbKey(fg)}|${rgbKey(bg)}`;
-
 const styleSeq = (fg, bg) => {
   if (fg && bg) return RESET + seq(fg, bg);
   if (fg) return RESET + ansi.fg(fg);
@@ -169,7 +167,7 @@ const overlayRange = (line, fromCol, toCol) => {
       fg = cell.bg ?? THEME.ctxBg;
       bg = cell.fg ?? THEME.ctxFg;
     }
-    const key = styleKey(fg, bg);
+    const key = `${rgbKey(fg)}|${rgbKey(bg)}`;
     if (key !== last) {
       out += styleSeq(fg, bg);
       last = key;
@@ -181,14 +179,11 @@ const overlayRange = (line, fromCol, toCol) => {
 
 const overlayRows = (rows, sel) => {
   if (!isRange(sel)) return rows;
-  const next = [];
-  for (let i = 0; i < rows.length; i++) {
+  const { length } = rows;
+  const next = new Array(length);
+  for (let i = 0; i < length; i++) {
     const range = lineRange(i, sel);
-    if (!range) {
-      next.push(rows[i]);
-      continue;
-    }
-    next.push(overlayRange(rows[i], range.from, range.to));
+    next[i] = range ? overlayRange(rows[i], range.from, range.to) : rows[i];
   }
   return next;
 };

package/lib/session/accessors.js

@@ -22,4 +22,12 @@ const bindGetters = (target, owner, names) => {
   }
 };
 
-module.exports = { bindAccessors, bindGetters, bindAlias };
+const stateView = (state, methods, writable = [], readonly = []) => {
+  const descriptors = Object.getOwnPropertyDescriptors(methods);
+  const view = Object.defineProperties({}, descriptors);
+  bindAccessors(view, state, writable);
+  bindGetters(view, state, readonly);
+  return view;
+};
+
+module.exports = { bindAccessors, bindGetters, bindAlias, stateView };

package/lib/session/actions.js

@@ -11,10 +11,9 @@ const ACTIONS = {
   feedback: ['f'],
   todo: ['t'],
   code: ['e'],
-  reload: ['r'],
   branch: ['b'],
-  pull: ['p'],
-  push: ['s'],
+  file: ['f'],
+  diff: ['d'],
   quit: ['q', 'ctrl-c'],
 };
 const ACTION_IDS = Object.keys(ACTIONS);
@@ -23,6 +22,16 @@ const BRANCH_ACTIONS = {
   newBranch: ['n'],
   rebase: ['r'],
   drop: ['d'],
+  pull: ['p'],
+  push: ['s'],
+};
+
+const COMMIT_ACTIONS = {
+  amend: ['a'],
+  fixup: ['f'],
+  drop: ['d'],
+  pull: ['p'],
+  push: ['s'],
 };
 
 const ACTION_HOT = {
@@ -40,8 +49,17 @@ const ACTION_LABEL = {
   next: '→',
 };
 
-const DIFF_DISABLED = ['commit', 'reload', 'branch', 'pull', 'push'];
+const DIFF_DISABLED = ['commit', 'branch', 'pull', 'push', 'file', 'diff'];
+const UNIT_DISABLED = [...DIFF_DISABLED, 'layout'];
 const FILES_DISABLED = ['layout', 'feedback', 'code'];
+const FILES_HIDDEN = [
+  ...FILES_DISABLED,
+  'prev',
+  'next',
+  'commit',
+  'file',
+  'diff',
+];
 const FILES_GIT_DISABLED = ['add', 'unstage', 'revert'];
 const FILES_TODO_DISABLED = [...FILES_GIT_DISABLED, ...FILES_DISABLED];
 const TODO_DISABLED = [
@@ -53,8 +71,9 @@ const TODO_DISABLED = [
   'feedback',
   'todo',
   'code',
-  'reload',
   'branch',
+  'file',
+  'diff',
   'pull',
   'push',
 ];
@@ -67,10 +86,25 @@ const BRANCHES_DISABLED = [
   'feedback',
   'todo',
   'code',
-  'reload',
   'branch',
-  'pull',
-  'push',
+  'file',
+  'diff',
+  'prev',
+  'next',
+];
+const COMMITS_DISABLED = [
+  'add',
+  'unstage',
+  'revert',
+  'layout',
+  'feedback',
+  'todo',
+  'code',
+  'branch',
+  'file',
+  'diff',
+  'prev',
+  'next',
 ];
 
 const isTodosTarget = (item) =>
@@ -82,17 +116,35 @@ const diffGitDisabled = (item) => {
   return ['unstage'];
 };
 
+const BRANCH_REMOTE = ['pull', 'push'];
+
 const branchGitDisabled = (item) => {
-  if (item && !item.current) return [];
-  return ['rebase', 'drop'];
+  const off = item && !item.current ? [] : ['rebase', 'drop'];
+  if (!item || !item.current) return [...off, ...BRANCH_REMOTE];
+  return off;
+};
+
+const commitWriteDisabled = (item) => {
+  if (!item || !item.sha) return ['amend', 'fixup', 'drop'];
+  return [];
+};
+
+const commitGitDisabled = (item) => {
+  const off = commitWriteDisabled(item);
+  if (item && item.canCommit === true) return off;
+  return [...off, 'commit', 'fixup'];
 };
 
 const disabledActions = (pane, item) => {
   const onTodos = isTodosTarget(item);
   if (pane === 'files') return onTodos ? FILES_TODO_DISABLED : FILES_DISABLED;
+  if (pane === 'unit') return [...UNIT_DISABLED, ...diffGitDisabled(item)];
   if (pane === 'branches') {
     return [...BRANCHES_DISABLED, ...branchGitDisabled(item)];
   }
+  if (pane === 'commits') {
+    return [...COMMITS_DISABLED, ...commitWriteDisabled(item)];
+  }
   if (onTodos) return TODO_DISABLED;
   return [...DIFF_DISABLED, ...diffGitDisabled(item)];
 };
@@ -116,16 +168,25 @@ const SCROLL = {
 };
 
 const KEY_TO_ACTION = {};
-const PANE_ONLY = ['branch', 'pull', 'push'];
+const PANE_ONLY = ['branch', 'file', 'diff'];
 const FILES_KEYS = {};
-for (const id of PANE_ONLY) {
-  for (const key of ACTIONS[id]) FILES_KEYS[key] = id;
+for (const key of ACTIONS.branch) FILES_KEYS[key] = 'branch';
+for (const id of BRANCH_REMOTE) {
+  for (const key of BRANCH_ACTIONS[id]) FILES_KEYS[key] = id;
 }
 const BRANCH_KEYS = {};
 for (const id of Object.keys(BRANCH_ACTIONS)) {
   for (const key of BRANCH_ACTIONS[id]) BRANCH_KEYS[key] = id;
 }
-const PANE_KEYS = { files: FILES_KEYS, branches: BRANCH_KEYS };
+const COMMIT_KEYS = {};
+for (const id of Object.keys(COMMIT_ACTIONS)) {
+  for (const key of COMMIT_ACTIONS[id]) COMMIT_KEYS[key] = id;
+}
+const PANE_KEYS = {
+  files: FILES_KEYS,
+  branches: BRANCH_KEYS,
+  commits: COMMIT_KEYS,
+};
 
 for (const id of ACTION_IDS) {
   if (PANE_ONLY.includes(id)) continue;
@@ -134,22 +195,28 @@ for (const id of ACTION_IDS) {
 
 for (const key of Object.keys(SCROLL)) KEY_TO_ACTION[key] = SCROLL[key];
 
-const actionFromKey = (key, pane) => {
+const actionFromKey = (key, pane, fileScope) => {
+  const listPane =
+    pane === 'files' || pane === 'branches' || pane === 'commits';
+  if (listPane && (key === 'left' || key === 'right')) return null;
   const paneKeys = PANE_KEYS[pane];
   if (paneKeys && paneKeys[key]) return paneKeys[key];
+  if (pane === 'files') {
+    if (key === 'f' && fileScope !== 'file') return 'file';
+    if (key === 'd' && fileScope === 'file') return 'diff';
+  }
   return KEY_TO_ACTION[key] ?? null;
 };
 
-const actionKeys = (id) => ACTIONS[id] ?? BRANCH_ACTIONS[id] ?? [];
+const actionKeys = (id) =>
+  ACTIONS[id] ?? BRANCH_ACTIONS[id] ?? COMMIT_ACTIONS[id] ?? [];
 
 const actionHot = (id) => ACTION_HOT[id] ?? actionKeys(id)[0];
 
-const actionLabel = (id) => ACTION_LABEL[id] ?? ACTION_HOT[id] ?? id;
-
 const actionLetter = actionHot;
 
 const buttonWord = (id) => {
-  const word = actionLabel(id);
+  const word = ACTION_LABEL[id] ?? ACTION_HOT[id] ?? id;
   const mark = actionLetter(id);
   if (word.includes(mark)) return word;
   return `${mark}${word}`;
@@ -173,19 +240,6 @@ const CONFIRM = {
       escape: 'cancel',
     },
   },
-  confirmCommit: {
-    choices: [
-      { letter: 'c', label: 'commit', lead: ' ' },
-      { letter: 'a', label: 'amend', lead: '  ' },
-      { letter: 'f', label: 'fixup', lead: '  ' },
-    ],
-    keys: {
-      c: 'commit',
-      a: 'amend',
-      f: 'fixup',
-      escape: 'cancel',
-    },
-  },
   confirmUpdate: {
     choices: YES_NO,
     prefix: (view) => {
@@ -220,13 +274,11 @@ const CONFIRM = {
 
 const CONFIRM_MODE = Object.keys(CONFIRM);
 
-const normalizeConfirmKey = (key) =>
-  key.length === 1 ? key.toLowerCase() : key;
-
 const confirmKey = (mode, key) => {
   const spec = CONFIRM[mode];
   if (!spec) return null;
-  return spec.keys[normalizeConfirmKey(key)] ?? null;
+  const norm = key.length === 1 ? key.toLowerCase() : key;
+  return spec.keys[norm] ?? null;
 };
 
 const applyConfirm = (mode, key, handlers) => {
@@ -248,19 +300,24 @@ module.exports = {
   ACTIONS,
   ACTION_IDS,
   BRANCH_ACTIONS,
+  COMMIT_ACTIONS,
   ACTION_HOT,
   ACTION_LABEL,
   DIFF_DISABLED,
+  UNIT_DISABLED,
   FILES_DISABLED,
+  FILES_HIDDEN,
   FILES_GIT_DISABLED,
   FILES_TODO_DISABLED,
   TODO_DISABLED,
   BRANCHES_DISABLED,
+  COMMITS_DISABLED,
   SCROLL,
   PANE_ONLY,
   disabledActions,
   diffGitDisabled,
   branchGitDisabled,
+  commitGitDisabled,
   actionFromKey,
   actionKeys,
   actionHot,