@hienlh/ppm 0.9.85 → 0.9.87

package/packages/ext-git-graph/src/git-log-parser.test.ts

@@ -0,0 +1,271 @@
+import { describe, it, expect } from "bun:test";
+import { parseGitLog } from "./git-log-parser.ts";
+import type { GitVertex } from "./types.ts";
+
+describe("git-log-parser: parseGitLog", () => {
+  it("parses single commit with minimal data", () => {
+    const input = `abc123
+def456
+John Doe
+john@example.com
+1609459200
+Jane Reviewer
+jane@example.com
+1609459200
+
+Initial commit
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    expect(result).toHaveLength(1);
+    const commit = result[0] as GitVertex;
+    expect(commit.hash).toBe("abc123");
+    expect(commit.parents).toEqual(["def456"]);
+    expect(commit.author).toBe("John Doe");
+    expect(commit.authorEmail).toBe("john@example.com");
+    expect(commit.authorDate).toBe(1609459200);
+    expect(commit.committer).toBe("Jane Reviewer");
+    expect(commit.committerEmail).toBe("jane@example.com");
+    expect(commit.commitDate).toBe(1609459200);
+    expect(commit.message).toBe("Initial commit");
+  });
+
+  it("parses commit with multiple parents (merge commit)", () => {
+    const input = `abc123
+def456 ghi789
+Author Name
+author@example.com
+1609459200
+Committer Name
+committer@example.com
+1609459200
+
+Merge branch 'feature'
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    expect(result).toHaveLength(1);
+    const commit = result[0] as GitVertex;
+    expect(commit.parents).toEqual(["def456", "ghi789"]);
+  });
+
+  it("parses commit with no parents (root commit)", () => {
+    const input = `abc123
+
+Author Name
+author@example.com
+1609459200
+Committer Name
+committer@example.com
+1609459200
+
+Root commit
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    expect(result).toHaveLength(1);
+    const commit = result[0] as GitVertex;
+    expect(commit.parents).toEqual([]);
+  });
+
+  it("parses refs with HEAD pointer", () => {
+    const input = `abc123
+def456
+Author
+author@example.com
+1609459200
+Committer
+committer@example.com
+1609459200
+HEAD -> main, origin/main
+Commit message
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    const commit = result[0] as GitVertex;
+    expect(commit.refs).toHaveLength(2);
+    expect(commit.refs[0]).toEqual({ name: "main", type: "head" });
+    expect(commit.refs[1]).toEqual({ name: "origin/main", type: "remote" });
+  });
+
+  it("parses refs with tags", () => {
+    const input = `abc123
+def456
+Author
+author@example.com
+1609459200
+Committer
+committer@example.com
+1609459200
+tag: v1.0.0, tag: latest
+Commit message
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    const commit = result[0] as GitVertex;
+    const tags = commit.refs.filter((r) => r.type === "tag");
+    expect(tags).toHaveLength(2);
+    expect(tags[0]).toEqual({ name: "v1.0.0", type: "tag" });
+    expect(tags[1]).toEqual({ name: "latest", type: "tag" });
+  });
+
+  it("parses refs with local branches", () => {
+    const input = `abc123
+def456
+Author
+author@example.com
+1609459200
+Committer
+committer@example.com
+1609459200
+feature-branch, develop
+Commit message
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    const commit = result[0] as GitVertex;
+    const locals = commit.refs.filter((r) => r.type === "local");
+    expect(locals).toHaveLength(2);
+    expect(locals[0]).toEqual({ name: "feature-branch", type: "local" });
+    expect(locals[1]).toEqual({ name: "develop", type: "local" });
+  });
+
+  it("parses commit with empty refs", () => {
+    const input = `abc123
+def456
+Author
+author@example.com
+1609459200
+Committer
+committer@example.com
+1609459200
+
+Commit with no refs
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    const commit = result[0] as GitVertex;
+    expect(commit.refs).toEqual([]);
+  });
+
+  it("parses multiple commits in sequence", () => {
+    const input = `abc123
+def456
+Author A
+authorA@example.com
+1609459200
+Committer A
+committerA@example.com
+1609459200
+
+First commit
+<END_COMMIT>def456
+ghi789
+Author B
+authorB@example.com
+1609459201
+Committer B
+committerB@example.com
+1609459201
+
+Second commit
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    expect(result).toHaveLength(2);
+    expect(result[0]?.hash).toBe("abc123");
+    expect(result[1]?.hash).toBe("def456");
+  });
+
+  it("handles multiline commit messages", () => {
+    const input = `abc123
+def456
+Author
+author@example.com
+1609459200
+Committer
+committer@example.com
+1609459200
+
+Fix: multiple fixes
+- Fix issue A
+- Fix issue B
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    const commit = result[0] as GitVertex;
+    // Only the first line of the message is captured (git log format)
+    expect(commit.message).toBe("Fix: multiple fixes");
+  });
+
+  it("filters out malformed blocks (less than 10 lines)", () => {
+    const input = `abc123
+def456
+Author
+author@example.com
+1609459200
+Committer
+committer@example.com
+1609459200
+
+Complete commit
+<END_COMMIT>incomplete
+block
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    expect(result).toHaveLength(1);
+  });
+
+  it("parses empty input as empty array", () => {
+    const result = parseGitLog("");
+    expect(result).toEqual([]);
+  });
+
+  it("handles whitespace-only input", () => {
+    const result = parseGitLog("   \n  \n  ");
+    expect(result).toEqual([]);
+  });
+
+  it("correctly parses unix timestamps as numbers", () => {
+    const input = `abc123
+def456
+Author
+author@example.com
+1609459200
+Committer
+committer@example.com
+1609459300
+
+Commit
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    const commit = result[0] as GitVertex;
+    expect(typeof commit.authorDate).toBe("number");
+    expect(typeof commit.commitDate).toBe("number");
+    expect(commit.authorDate).toBe(1609459200);
+    expect(commit.commitDate).toBe(1609459300);
+  });
+
+  it("parses refs with remote branches (origin/feature)", () => {
+    const input = `abc123
+def456
+Author
+author@example.com
+1609459200
+Committer
+committer@example.com
+1609459200
+origin/feature, upstream/main
+Commit
+<END_COMMIT>`;
+
+    const result = parseGitLog(input);
+    const commit = result[0] as GitVertex;
+    const remotes = commit.refs.filter((r) => r.type === "remote");
+    expect(remotes).toHaveLength(2);
+    expect(remotes[0]).toEqual({ name: "origin/feature", type: "remote" });
+    expect(remotes[1]).toEqual({ name: "upstream/main", type: "remote" });
+  });
+});

package/packages/ext-git-graph/src/git-log-parser.ts

@@ -0,0 +1,38 @@
+/**
+ * Parse git log output into structured GitVertex objects.
+ * Uses custom format: %H%n%P%n%an%n%ae%n%at%n%cn%n%ce%n%ct%n%D%n%s%n<END_COMMIT>
+ */
+import type { GitVertex, RefData } from "./types.ts";
+
+export function parseGitLog(stdout: string): GitVertex[] {
+  const blocks = stdout.split("<END_COMMIT>").filter((b) => b.trim());
+  return blocks.map(parseCommitBlock).filter(Boolean) as GitVertex[];
+}
+
+function parseCommitBlock(block: string): GitVertex | null {
+  const lines = block.trim().split("\n");
+  if (lines.length < 10) return null;
+
+  const hash = lines[0];
+  const parents = lines[1] ? lines[1].split(" ").filter(Boolean) : [];
+  const author = lines[2];
+  const authorEmail = lines[3];
+  const authorDate = parseInt(lines[4], 10);
+  const committer = lines[5];
+  const committerEmail = lines[6];
+  const commitDate = parseInt(lines[7], 10);
+  const refs = parseRefs(lines[8]);
+  const message = lines[9];
+
+  return { hash, parents, author, authorEmail, authorDate, committer, committerEmail, commitDate, refs, message };
+}
+
+function parseRefs(refString: string): RefData[] {
+  if (!refString.trim()) return [];
+  return refString.split(",").map((r) => r.trim()).filter(Boolean).map((ref) => {
+    if (ref.startsWith("HEAD -> ")) return { name: ref.replace("HEAD -> ", ""), type: "head" as const };
+    if (ref.startsWith("tag: ")) return { name: ref.replace("tag: ", ""), type: "tag" as const };
+    if (ref.includes("/")) return { name: ref, type: "remote" as const };
+    return { name: ref, type: "local" as const };
+  });
+}

package/packages/ext-git-graph/src/types.ts

@@ -0,0 +1,192 @@
+/** Message types for Extension ↔ Webview communication */
+
+// --- Git data types ---
+
+export interface GitVertex {
+  hash: string;
+  parents: string[];
+  author: string;
+  authorEmail: string;
+  authorDate: number;
+  committer: string;
+  committerEmail: string;
+  commitDate: number;
+  refs: RefData[];
+  message: string;
+}
+
+export interface RefData {
+  name: string;
+  type: "head" | "local" | "remote" | "tag";
+}
+
+export interface Branch {
+  name: string;
+  remote?: string;
+  current: boolean;
+  hash: string;
+}
+
+export interface Tag {
+  name: string;
+  hash: string;
+}
+
+export interface Remote {
+  name: string;
+  fetchUrl: string;
+  pushUrl: string;
+}
+
+export interface Stash {
+  index: number;
+  hash: string;
+  message: string;
+}
+
+export interface Worktree {
+  path: string;
+  branch: string;
+  head: string;
+  isMain: boolean;
+  isDetached: boolean;
+  locked: boolean;
+  lockReason?: string;
+  prunable: boolean;
+}
+
+export interface CommitDetail {
+  hash: string;
+  author: string;
+  authorEmail: string;
+  authorDate: number;
+  committer: string;
+  committerEmail: string;
+  commitDate: number;
+  message: string;
+  parents: string[];
+  fileChanges: FileChange[];
+}
+
+export interface FileChange {
+  path: string;
+  oldPath?: string;
+  status: "A" | "M" | "D" | "R" | "C" | "U";
+  additions: number;
+  deletions: number;
+}
+
+export interface MergeState {
+  type: "merge" | "rebase" | "cherry-pick";
+  progress?: string; // e.g. "3/5" for rebase
+  message?: string;  // current commit message being rebased
+}
+
+export interface RepoInfo {
+  path: string;
+  branches: Branch[];
+  tags: Tag[];
+  remotes: Remote[];
+  stashes: Stash[];
+  head: string;
+  currentBranch: string;
+}
+
+export interface ActionResult {
+  ok: boolean;
+  error?: string;
+}
+
+export interface UncommittedData {
+  staged: FileChange[];
+  unstaged: FileChange[];
+  conflicted: FileChange[];
+  mergeState?: MergeState;
+}
+
+// --- Settings ---
+
+export interface IssueLinkingRule {
+  pattern: string;
+  url: string;
+}
+
+export interface PrCreationConfig {
+  provider: "github" | "gitlab" | "bitbucket" | "custom";
+  urlTemplate: string;
+  owner: string;
+  repo: string;
+  defaultTargetBranch: string;
+}
+
+export interface GitGraphSettings {
+  maxCommits: number;
+  showTags: boolean;
+  showStashes: boolean;
+  showRemoteBranches: boolean;
+  graphStyle: "rounded" | "angular";
+  firstParentOnly: boolean;
+  dateFormat: "relative" | "absolute" | "iso";
+  commitOrdering: "topo" | "date" | "author-date";
+  issueLinkingRules: IssueLinkingRule[];
+  prCreation: PrCreationConfig | null;
+  autoFetchInterval: number;
+}
+
+export const DEFAULT_SETTINGS: GitGraphSettings = {
+  maxCommits: 300,
+  showTags: true,
+  showStashes: true,
+  showRemoteBranches: true,
+  graphStyle: "rounded",
+  firstParentOnly: false,
+  dateFormat: "relative",
+  commitOrdering: "topo",
+  issueLinkingRules: [{ pattern: "#(\\d+)", url: "" }],
+  prCreation: null,
+  autoFetchInterval: 0,
+};
+
+// --- Extension → Webview messages ---
+
+export type ExtToWebview =
+  | { command: "loadRepoInfo"; data: RepoInfo }
+  | { command: "loadCommits"; data: GitVertex[]; append: boolean }
+  | { command: "commitDetails"; data: CommitDetail }
+  | { command: "loadUncommitted"; data: UncommittedData | null }
+  | { command: "loadSettings"; data: GitGraphSettings }
+  | { command: "loadUserDetails"; data: { name: string; email: string } }
+  | { command: "loadOwnerRepo"; data: { owner: string; repo: string } }
+  | { command: "refresh"; data: GitVertex[]; repoInfo: RepoInfo }
+  | { command: "actionResult"; action: string; args?: Record<string, unknown>; result: ActionResult }
+  | { command: "loadWorktrees"; data: Worktree[] }
+  | { command: "loadStashes"; data: Stash[] }
+  | { command: "error"; message: string };
+
+// --- Webview → Extension messages ---
+
+export type WebviewToExt =
+  | { command: "ready" }
+  | { command: "requestRepoInfo" }
+  | { command: "requestCommits"; maxCommits?: number; skip?: number; branch?: string }
+  | { command: "requestCommitDetails"; hash: string }
+  | { command: "requestUncommitted" }
+  | { command: "openDiff"; filePath: string; hash: string; parentHash: string | null }
+  | { command: "requestSettings" }
+  | { command: "updateSetting"; key: string; value: unknown }
+  | { command: "requestUserDetails" }
+  | { command: "updateUserDetails"; name?: string; email?: string }
+  | { command: "addRemote"; name: string; url: string }
+  | { command: "removeRemote"; name: string }
+  | { command: "editRemoteUrl"; name: string; url: string }
+  | { command: "requestOwnerRepo" }
+  | { command: "gitAction"; action: string; args: Record<string, unknown> }
+  | { command: "requestWorktrees" }
+  | { command: "addWorktree"; path: string; branch?: string; newBranch?: string; startPoint?: string }
+  | { command: "removeWorktree"; path: string; force?: boolean }
+  | { command: "pruneWorktrees" }
+  | { command: "openWorktree"; path: string }
+  | { command: "openFile"; filePath: string }
+  | { command: "openConflictFile"; filePath: string }
+  | { command: "openSourceControl" }
+  | { command: "requestStashes" };

package/packages/ext-git-graph/src/webview-html.test.ts

@@ -0,0 +1,142 @@
+import { describe, it, expect } from "bun:test";
+import { getWebviewHtml } from "./webview-html.ts";
+
+describe("webview-html: getWebviewHtml", () => {
+  it("returns valid HTML", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain("<!DOCTYPE html>");
+    expect(html).toContain("</html>");
+  });
+
+  it("includes essential elements", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain('<div id="app">');
+    expect(html).toContain('<header id="toolbar">');
+    expect(html).toContain('<div id="graph-container">');
+    expect(html).toContain('<div id="detail-panel"');
+    expect(html).toContain('<div id="status-bar">');
+    expect(html).toContain('<div id="context-menu"');
+  });
+
+  it("includes find bar", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain('id="find-bar"');
+    expect(html).toContain('id="find-input"');
+    expect(html).toContain('id="find-count"');
+    expect(html).toContain('id="find-prev"');
+    expect(html).toContain('id="find-next"');
+    expect(html).toContain('id="find-close"');
+  });
+
+  it("includes toolbar buttons", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain('id="branch-selector"');
+    expect(html).toContain('id="btn-refresh"');
+    expect(html).toContain('id="btn-find"');
+    expect(html).toContain('id="btn-settings"');
+  });
+
+  it("includes commit list columns", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain("col-graph");
+    expect(html).toContain("col-message");
+    expect(html).toContain("col-author");
+    expect(html).toContain("col-date");
+    expect(html).toContain("col-hash");
+  });
+
+  it("includes CSS styles", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain("<style>");
+    expect(html).toContain("</style>");
+    expect(html).toContain("--bg:");
+    expect(html).toContain("--text:");
+    expect(html).toContain("--border:");
+  });
+
+  it("includes dark mode styles", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain("prefers-color-scheme: dark");
+  });
+
+  it("includes JavaScript", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain("<script>");
+    expect(html).toContain("</script>");
+  });
+
+  it("includes graph container and commit list", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain('id="graph-header"');
+    expect(html).toContain('id="commit-list"');
+    expect(html).toContain('id="loading"');
+    expect(html).toContain('id="graph-svg-container"');
+    expect(html).toContain('id="commit-list-wrapper"');
+  });
+
+  it("marks elements with proper classes", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain("hidden");
+    expect(html).toContain("commit-row");
+    expect(html).toContain("header-row");
+  });
+
+  it("includes viewport meta tag", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain('meta charset="utf-8"');
+  });
+
+  it("includes responsive flex layout", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain("flex");
+    expect(html).toContain("flex-direction");
+  });
+
+  it("sets initial status text", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain("Loading repository");
+  });
+
+  it("includes SVG graph rendering capability (comment)", () => {
+    const html = getWebviewHtml();
+    // Graph rendering would be in the JavaScript section
+    expect(html).toContain("<script>");
+  });
+
+  it("includes CSS variables for theming", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain("--blue:");
+    expect(html).toContain("--red:");
+    expect(html).toContain("--green:");
+    expect(html).toContain("--yellow:");
+    expect(html).toContain("--purple:");
+    expect(html).toContain("--orange:");
+  });
+
+  it("includes graph column width variable", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain("--graph-col-w");
+  });
+
+  it("includes overflow handling for containers", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain("overflow");
+  });
+
+  it("is valid HTML structure", () => {
+    const html = getWebviewHtml();
+    // Check nesting: html > body > div#app
+    const bodyStart = html.indexOf("<body>");
+    const bodyEnd = html.indexOf("</body>");
+    const appDiv = html.indexOf('id="app"');
+    expect(bodyStart).toBeGreaterThan(-1);
+    expect(bodyEnd).toBeGreaterThan(bodyStart);
+    expect(appDiv).toBeGreaterThan(bodyStart);
+    expect(appDiv).toBeLessThan(bodyEnd);
+  });
+
+  it("includes proper charset declaration", () => {
+    const html = getWebviewHtml();
+    expect(html).toContain('charset="utf-8"');
+  });
+});