reslop 0.1.9 → 0.2.0

package/lib/remote/headers.js

@@ -1,36 +0,0 @@
-'use strict';
-
-const headerValue = (headers, name) => {
-  if (!headers) return '';
-  if (typeof headers.get === 'function') {
-    return headers.get(name) || '';
-  }
-  const wanted = name.toLowerCase();
-  for (const key of Object.keys(headers)) {
-    const value = headers[key];
-    if (key.toLowerCase() === wanted) return `${value}`;
-  }
-  return '';
-};
-
-const parseLinkNext = (headers) => {
-  const link = headerValue(headers, 'link');
-  if (!link) return '';
-  for (const part of link.split(',')) {
-    const match = /<([^>]+)>\s*;\s*rel="?next"?/i.exec(part.trim());
-    if (match) return match[1];
-  }
-  return '';
-};
-
-const parseNextPage = (headers, currentUrl) => {
-  const fromLink = parseLinkNext(headers);
-  if (fromLink) return fromLink;
-  const page = headerValue(headers, 'x-next-page');
-  if (!page) return '';
-  const url = new URL(currentUrl);
-  url.searchParams.set('page', page);
-  return url.toString();
-};
-
-module.exports = { headerValue, parseLinkNext, parseNextPage };

package/lib/remote/http.js

@@ -1,122 +0,0 @@
-'use strict';
-
-const { delay, isHashObject } = require('metautil');
-
-const RETRY_ATTEMPTS = 3;
-const RETRY_DELAY_MS = 200;
-const TRANSIENT_STATUS = [429, 502, 503, 504];
-
-const isRecord = isHashObject;
-
-const oneLine = (text) => {
-  const value = `${text ?? ''}`.trim();
-  if (!value) return '';
-  return value.split('\n')[0];
-};
-
-const abortError = (signal) => {
-  if (signal && signal.reason !== undefined) return signal.reason;
-  const error = new Error('This operation was aborted');
-  error.name = 'AbortError';
-  return error;
-};
-
-const isAbortError = (error, signal) => {
-  if (signal && signal.aborted) return true;
-  if (!error || typeof error !== 'object') return false;
-  if (error.name === 'AbortError') return true;
-  if (error.code === 'ABORT') return true;
-  return false;
-};
-
-const throwIfAborted = (signal) => {
-  if (signal && signal.aborted) throw abortError(signal);
-};
-
-const createTransport = (deps = {}) => {
-  const fetchImpl = deps.fetch;
-  const rawDelay = deps.delay ?? delay;
-  const delayFn = async (ms, abortSignal) => {
-    try {
-      await rawDelay(ms, abortSignal);
-    } catch (error) {
-      throwIfAborted(abortSignal);
-      throw error;
-    }
-  };
-  const signal = deps.signal;
-
-  const retry = async (run, options = {}) => {
-    const attempts = options.attempts ?? RETRY_ATTEMPTS;
-    const delayMs = options.delayMs ?? RETRY_DELAY_MS;
-    const isRetryable = options.isRetryable;
-    for (let i = 0; ; i++) {
-      try {
-        return await run();
-      } catch (error) {
-        if (isAbortError(error, signal)) throw error;
-        const isLast = i === attempts - 1;
-        if (isLast || !isRetryable(error)) throw error;
-        throwIfAborted(signal);
-        await delayFn(delayMs * (i + 1), signal);
-      }
-    }
-  };
-
-  const request = async (url, options = {}) => {
-    throwIfAborted(signal);
-    if (typeof fetchImpl !== 'function') {
-      throw options.missingFetch();
-    }
-    const headers = options.headers;
-    const method = options.method || 'GET';
-    const body = options.body;
-    const wrapNetwork = options.wrapNetwork;
-    const toHttpError = options.toHttpError;
-    const retryOpts = options.retry ?? {};
-    return retry(
-      async () => {
-        let response;
-        try {
-          const init = { method, headers };
-          if (body !== undefined) init.body = body;
-          if (signal) init.signal = signal;
-          response = await fetchImpl(url, init);
-        } catch (error) {
-          if (isAbortError(error, signal)) throw error;
-          throw wrapNetwork(error);
-        }
-        const text = await response.text();
-        if (!response.ok) throw toHttpError(response.status, text);
-        return { body: text, headers: response.headers };
-      },
-      { ...retryOpts, isRetryable: options.isRetryable },
-    );
-  };
-
-  const listPages = async (url, options = {}) => {
-    const items = [];
-    let next = url;
-    const parsePage = options.parsePage;
-    const nextPage = options.nextPage;
-    while (next) {
-      const page = await request(next, options);
-      const entries = parsePage(page.body);
-      for (const entry of entries) items.push(entry);
-      next = nextPage(page.headers, next);
-    }
-    return items;
-  };
-
-  return { retry, request, listPages };
-};
-
-module.exports = {
-  RETRY_ATTEMPTS,
-  RETRY_DELAY_MS,
-  TRANSIENT_STATUS,
-  isRecord,
-  oneLine,
-  isAbortError,
-  createTransport,
-};

package/lib/remote/notes.js

@@ -1,94 +0,0 @@
-'use strict';
-
-const { isCtxType } = require('../diff.js');
-const { itemPath } = require('../files.js');
-
-const sideLineNumber = (type, oldNo, newNo, side) => {
-  if (side === 'LEFT') {
-    if (type === 'add') return 0;
-    return oldNo;
-  }
-  if (type === 'del') return 0;
-  return newNo;
-};
-
-const locateLineInItem = (item, rel, line, side) => {
-  const path = itemPath(item);
-  if (path !== rel) return null;
-  const hunk = item.hunk;
-  if (!hunk || !line) return null;
-  let oldNo = hunk.oldStart;
-  let newNo = hunk.newStart;
-  for (const entry of hunk.lines) {
-    const at = sideLineNumber(entry.type, oldNo, newNo, side);
-    if (at === line) {
-      const inBlock = entry.blockId === item.blockId;
-      return { inBlock };
-    }
-    if (isCtxType(entry.type) || entry.type === 'del') {
-      oldNo += 1;
-    }
-    if (isCtxType(entry.type) || entry.type === 'add') {
-      newNo += 1;
-    }
-  }
-  return null;
-};
-
-const findItemForLine = (items, rel, line, side) => {
-  let fallback = null;
-  for (const item of items) {
-    const hit = locateLineInItem(item, rel, line, side);
-    if (!hit) continue;
-    if (hit.inBlock) return item;
-    if (!fallback) fallback = item;
-  }
-  return fallback;
-};
-
-const importedText = (entry, host) => {
-  const reviewer = entry.reviewer || 'unknown';
-  const body = `${entry.body ?? ''}`.trim();
-  return `@${reviewer} review at ${host}: ${body}`;
-};
-
-const importedTodo = (file, entry, host) => ({
-  kind: 'todo',
-  file,
-  text: importedText(entry, host),
-  done: entry.resolved === true,
-});
-
-const importedFeedback = (item, path, entry, host) => {
-  const hunk = item.hunk;
-  return {
-    kind: 'feedback',
-    file: itemPath(item) || path,
-    oldStart: hunk ? hunk.oldStart : 0,
-    newStart: hunk ? hunk.newStart : 0,
-    blockId: item.blockId ?? 0,
-    origin: item.origin ?? 'pr',
-    header: hunk ? hunk.header : '',
-    text: importedText(entry, host),
-    done: entry.resolved === true,
-  };
-};
-
-const noteFromLocation = (items, loc, entry, host, fallbackFile) => {
-  const path = loc.path || '';
-  const line = loc.line;
-  const side = loc.side;
-  if (line && path) {
-    const item = findItemForLine(items, path, line, side);
-    if (item) return importedFeedback(item, path, entry, host);
-  }
-  if (path) return importedTodo(path, entry, host);
-  return importedTodo(fallbackFile, entry, host);
-};
-
-module.exports = {
-  findItemForLine,
-  importedText,
-  importedTodo,
-  noteFromLocation,
-};

package/lib/review/format.js

@@ -1,434 +0,0 @@
-'use strict';
-
-const path = require('node:path');
-
-const { TODO_FILE } = require('../files.js');
-const store = require('./store.js');
-const { createStore, feedbackKey } = store;
-
-const REVIEW_STATUSES = ['editing', 'ready', 'partial', 'done'];
-const FEED_HEAD = /^### Feedback `([^`]+)` \+(\d+) \((.*)\)\s*$/;
-const FEED_BLOCK = /^(.*), block (\d+)$/;
-const FEED_BLOCK_ONLY = /^block (\d+)$/;
-const FEED_META = /^<!-- reslop:(.*) -->\s*$/;
-const FEED_SUFFIX = /^(.*) - (.+):(\d+):(\d+):(\d+)$/;
-const CHECK_ITEM = /^- \[([ xX])\] (.*)$/;
-const CODE_HEAD = /^- \[([ xX])\] code `([^`]+)`\s*$/;
-const FENCE_LINE = /^(`{3,})\s*$/;
-const FILE_QUOTE = /^> (.+)$/;
-const STATUS_LINE = /^status:\s*(\S+)\s*$/;
-const GIT_ORIGINS = ['unstaged', 'staged', 'untracked', 'commit', 'pr'];
-const SECTION_LINE = {
-  '### Todo': 'todo',
-  '### Feedback': 'feedback',
-};
-
-const AGENT_INSTRUCTIONS = [
-  'Execute reviews with `status: ready` or `status: partial`.',
-  'Do not start `editing` or `done` review files.',
-  'Work through unchecked items. After finishing one, mark it `[x]`.',
-  'For `code` items, replace added lines at that location with the fence.',
-  'Set `status` to `partial` if some remain, or `done` if all are `[x]`.',
-  'Do not redo items already marked `[x]`.',
-].join('\n');
-
-const toInt = (text) => parseInt(text, 10);
-
-const canonicalStatus = (raw) => {
-  if (!raw) return '';
-  const name = raw === 'pending' ? 'ready' : raw;
-  if (REVIEW_STATUSES.includes(name)) return name;
-  return '';
-};
-
-const parseFrontmatterStatus = (text) => {
-  if (!text.startsWith('---')) return '';
-  const nl = text.indexOf('\n');
-  if (nl < 0) return '';
-  const rest = text.slice(nl + 1);
-  const end = rest.indexOf('\n---');
-  if (end < 0) return '';
-  const matter = rest.slice(0, end);
-  for (const line of matter.split('\n')) {
-    const match = STATUS_LINE.exec(line);
-    if (match) return canonicalStatus(match[1]);
-  }
-  return '';
-};
-
-const isDoneMark = (mark) => mark === 'x' || mark === 'X';
-
-const pushTodo = (builder, file, text, done) => {
-  const id = builder.nextTodoId;
-  builder.nextTodoId += 1;
-  builder.todos.push({ id, file, text, done });
-};
-
-const checkboxLine = (text, done) => {
-  const flat = text.trim().replace(/\n/g, ' ');
-  const mark = done ? 'x' : ' ';
-  return `- [${mark}] ${flat}`;
-};
-
-const storeFeedback = (builder, note, key) => {
-  const next = {
-    file: note.file,
-    oldStart: note.oldStart ?? 0,
-    newStart: note.newStart ?? 0,
-    blockId: note.blockId ?? 0,
-    origin: note.origin ?? '',
-    header: note.header ?? '',
-    text: note.text,
-    done: note.done ?? false,
-  };
-  builder.feedback.set(key || feedbackKey(next), next);
-};
-
-const storeCode = (builder, note, key) => {
-  const next = {
-    file: note.file,
-    oldStart: note.oldStart ?? 0,
-    newStart: note.newStart ?? 0,
-    blockId: note.blockId ?? 0,
-    origin: note.origin ?? '',
-    header: note.header ?? '',
-    text: note.text,
-    done: note.done ?? false,
-  };
-  builder.code.set(key || feedbackKey(next), next);
-};
-
-const feedbackLine = (note) => {
-  const loc = feedbackKey(note);
-  return `${checkboxLine(note.text, note.done)} - ${loc}`;
-};
-
-const parseLoc = (raw) => {
-  const parts = raw.split(':');
-  const n = parts.length;
-  if (n < 4) return null;
-  const blockId = toInt(parts[n - 1]);
-  const newStart = toInt(parts[n - 2]);
-  const oldStart = toInt(parts[n - 3]);
-  if (!Number.isFinite(blockId + newStart + oldStart)) return null;
-  let fileParts = parts.slice(0, n - 3);
-  let origin = '';
-  if (fileParts.length > 1 && GIT_ORIGINS.includes(fileParts[0])) {
-    origin = fileParts[0];
-    fileParts = fileParts.slice(1);
-  }
-  if (!fileParts.length) return null;
-  const file = fileParts.join(':');
-  return { origin, file, oldStart, newStart, blockId };
-};
-
-const parseFeedSuffix = (textItem) => {
-  const match = FEED_SUFFIX.exec(textItem);
-  if (!match) return null;
-  const text = match[1];
-  const file = match[2];
-  const oldStart = toInt(match[3]);
-  const newStart = toInt(match[4]);
-  const blockId = toInt(match[5]);
-  return { text, file, oldStart, newStart, blockId };
-};
-
-const fileGroup = (groups, file) => {
-  let group = groups.get(file);
-  if (!group) {
-    group = { todos: [], feedback: [], code: [] };
-    groups.set(file, group);
-  }
-  return group;
-};
-
-const groupReviewNotes = (notes) => {
-  const groups = new Map();
-  for (const todo of notes.todos) {
-    if (!todo.text.trim()) continue;
-    fileGroup(groups, TODO_FILE).todos.push(todo);
-  }
-  for (const key of notes.feedback.keys()) {
-    const note = notes.feedback.get(key);
-    if (!note.text.trim()) continue;
-    fileGroup(groups, note.file).feedback.push({ key, note });
-  }
-  for (const key of notes.code.keys()) {
-    const note = notes.code.get(key);
-    fileGroup(groups, note.file).code.push({ key, note });
-  }
-  return groups;
-};
-
-const reviewHeader = (notes) => {
-  const name = path.basename(notes.reviewPath, '.md');
-  const status = canonicalStatus(notes.status) || 'editing';
-  return [
-    '---',
-    `status: ${status}`,
-    '---',
-    '',
-    `# reslop review ${name}`,
-    '',
-    '## Agent instructions',
-    '',
-    AGENT_INSTRUCTIONS,
-    '',
-  ];
-};
-
-const fenceFor = (text) => {
-  let n = 3;
-  for (const line of text.split('\n')) {
-    const match = /^(`{3,})/.exec(line);
-    if (!match) continue;
-    if (match[1].length >= n) n = match[1].length + 1;
-  }
-  return '`'.repeat(n);
-};
-
-const emitCodeItem = (lines, note) => {
-  const loc = feedbackKey(note);
-  const mark = note.done ? 'x' : ' ';
-  const fence = fenceFor(note.text);
-  lines.push(`- [${mark}] code \`${loc}\``);
-  lines.push(fence);
-  if (note.text !== '') lines.push(note.text);
-  lines.push(fence);
-};
-
-const emitReviewGroups = (lines, groups) => {
-  for (const file of groups.keys()) {
-    const group = groups.get(file);
-    lines.push(`## ${file}`);
-    lines.push('');
-    for (const todo of group.todos) {
-      lines.push(checkboxLine(todo.text, todo.done));
-    }
-    for (const item of group.feedback) {
-      lines.push(feedbackLine(item.note));
-    }
-    for (const item of group.code) {
-      emitCodeItem(lines, item.note);
-    }
-    lines.push('');
-  }
-};
-
-const serializeReview = (notes) => {
-  const lines = reviewHeader(notes);
-  emitReviewGroups(lines, groupReviewNotes(notes));
-  return `${lines.join('\n')}\n`;
-};
-
-const parseFeedbackHeading = (line) => {
-  const match = FEED_HEAD.exec(line);
-  if (!match) return null;
-  const file = match[1];
-  const newStart = toInt(match[2]);
-  const rest = match[3];
-  const only = FEED_BLOCK_ONLY.exec(rest);
-  if (only) {
-    return { file, newStart, header: '', blockId: toInt(only[1]) };
-  }
-  const withHead = FEED_BLOCK.exec(rest);
-  if (!withHead) {
-    return { file, newStart, header: rest, blockId: 0 };
-  }
-  const header = withHead[1];
-  const blockId = toInt(withHead[2]);
-  return { file, newStart, header, blockId };
-};
-
-const flushPendingFeedback = (builder, pending) => {
-  if (!pending || !(pending.text ?? '').trim()) return null;
-  storeFeedback(builder, pending, pending.key);
-  return null;
-};
-
-const headingFile = (line) => {
-  const quoted = FILE_QUOTE.exec(line);
-  if (quoted) return quoted[1];
-  if (line.startsWith('## ') && line !== '## Agent instructions') {
-    return line.slice(3);
-  }
-  return null;
-};
-
-const startFeedbackPending = (heading, file) => ({
-  file: heading.file || file,
-  newStart: heading.newStart,
-  header: heading.header,
-  blockId: heading.blockId,
-  oldStart: heading.newStart,
-  origin: '',
-  key: '',
-  text: '',
-});
-
-const applyFeedMeta = (pending, raw) => {
-  const loc = parseLoc(raw);
-  if (!loc) return pending;
-  pending.origin = loc.origin;
-  pending.file = loc.file;
-  pending.oldStart = loc.oldStart;
-  pending.newStart = loc.newStart;
-  pending.blockId = loc.blockId;
-  pending.key = feedbackKey(loc);
-  return pending;
-};
-
-const applyCheckItem = (builder, file, section, pending, textItem, done) => {
-  if (section === 'todo' && file) {
-    pushTodo(builder, file, textItem, done);
-    return pending;
-  }
-  const parsed = parseFeedSuffix(textItem);
-  if (parsed) {
-    flushPendingFeedback(builder, pending);
-    storeFeedback(builder, {
-      file: parsed.file || file,
-      oldStart: parsed.oldStart,
-      newStart: parsed.newStart,
-      blockId: parsed.blockId,
-      origin: '',
-      header: '',
-      text: parsed.text,
-      done,
-    });
-    return null;
-  }
-  if (pending) {
-    pending.text = textItem;
-    pending.done = done;
-    return flushPendingFeedback(builder, pending);
-  }
-  if (file && section !== 'feedback') {
-    pushTodo(builder, file, textItem, done);
-  }
-  return pending;
-};
-
-const applyReviewLine = (builder, file, section, pending, line) => {
-  const nextFile = headingFile(line);
-  if (nextFile !== null) {
-    pending = flushPendingFeedback(builder, pending);
-    return { file: nextFile, section: '', pending };
-  }
-  const nextSection = SECTION_LINE[line];
-  if (nextSection) {
-    pending = flushPendingFeedback(builder, pending);
-    return { file, section: nextSection, pending };
-  }
-  const heading = parseFeedbackHeading(line);
-  if (heading) {
-    flushPendingFeedback(builder, pending);
-    pending = startFeedbackPending(heading, file);
-    return { file, section: 'feedback', pending };
-  }
-  const meta = FEED_META.exec(line);
-  if (meta && pending) {
-    pending = applyFeedMeta(pending, meta[1]);
-    return { file, section, pending };
-  }
-  const item = CHECK_ITEM.exec(line);
-  if (!item) return { file, section, pending };
-  const done = isDoneMark(item[1]);
-  pending = applyCheckItem(builder, file, section, pending, item[2], done);
-  return { file, section, pending };
-};
-
-const startCodePending = (line) => {
-  const match = CODE_HEAD.exec(line);
-  if (!match) return null;
-  const loc = parseLoc(match[2]);
-  if (!loc) return null;
-  return {
-    done: isDoneMark(match[1]),
-    origin: loc.origin,
-    file: loc.file,
-    oldStart: loc.oldStart,
-    newStart: loc.newStart,
-    blockId: loc.blockId,
-    open: '',
-    lines: [],
-  };
-};
-
-const storeCodeNote = (builder, pending, text) => {
-  storeCode(builder, {
-    file: pending.file,
-    oldStart: pending.oldStart,
-    newStart: pending.newStart,
-    blockId: pending.blockId,
-    origin: pending.origin,
-    header: '',
-    text,
-    done: pending.done,
-  });
-};
-
-const flushPendingCode = (builder, pending) => {
-  if (!pending) return;
-  const text = pending.open ? pending.lines.join('\n') : '';
-  storeCodeNote(builder, pending, text);
-};
-
-const takeCodeLine = (builder, pending, line) => {
-  const fence = FENCE_LINE.exec(line);
-  if (!pending.open) {
-    if (fence) {
-      pending.open = fence[1];
-      return pending;
-    }
-    if (!line.trim()) return pending;
-    storeCodeNote(builder, pending, '');
-    return 'fallthrough';
-  }
-  if (fence && fence[1].length >= pending.open.length) {
-    storeCodeNote(builder, pending, pending.lines.join('\n'));
-    return null;
-  }
-  pending.lines.push(line);
-  return pending;
-};
-
-const parseReview = (text, reviewPath, templates = []) => {
-  const builder = createStore(reviewPath, templates);
-  builder.status = parseFrontmatterStatus(text) || 'editing';
-  let file = '';
-  let section = '';
-  let pending = null;
-  let pendingCode = null;
-  for (const line of text.split('\n')) {
-    if (pendingCode) {
-      const nextCode = takeCodeLine(builder, pendingCode, line);
-      if (nextCode === 'fallthrough') {
-        pendingCode = null;
-      } else {
-        pendingCode = nextCode;
-        continue;
-      }
-    }
-    const started = startCodePending(line);
-    if (started) {
-      pending = flushPendingFeedback(builder, pending);
-      pendingCode = started;
-      continue;
-    }
-    const next = applyReviewLine(builder, file, section, pending, line);
-    file = next.file;
-    section = next.section;
-    pending = next.pending;
-  }
-  flushPendingCode(builder, pendingCode);
-  flushPendingFeedback(builder, pending);
-  builder.dirty = false;
-  return builder;
-};
-
-module.exports = {
-  parseFrontmatterStatus,
-  serializeReview,
-  parseReview,
-};