Skip to content

changed comment a bit#28

Merged
DeDuckProject merged 2 commits intomainfrom
some-tweaks
Mar 13, 2026
Merged

changed comment a bit#28
DeDuckProject merged 2 commits intomainfrom
some-tweaks

Conversation

@DeDuckProject
Copy link
Copy Markdown
Owner

No description provided.

@DeDuckProject
Copy link
Copy Markdown
Owner Author

/glimpse

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 13, 2026

🧐 UI Demo Preview

Changes detected in: CLAUDE.md, CONTRIBUTING.md, README.md, packages/core/src/publisher/github-comment.ts

What changed: The documentation and setup experience for GitGlimpse has been significantly simplified, removing the dual-workflow /glimpse comment trigger system in favor of a straightforward pull request-only workflow. The README now shows a real example screenshot, streamlined YAML config, and a cleaner project structure reflecting new framework-specific route detection.

Demo

📱 Can't see the preview? Open it directly

Demo script (auto-generated)
import type { Page } from '@playwright/test';

export async function demo(page: Page): Promise<void> {
  await page.goto('http://localhost:3000', { waitUntil: 'networkidle' });
  await page.waitForLoadState('networkidle');
  await page.waitForTimeout(1500);

  // Scroll to top to show the example PR comment screenshot
  await page.evaluate(() => window.scrollTo({ top: 0, behavior: 'smooth' }));
  await page.waitForTimeout(1500);

  // Look for the example comment image or heading at the top of README
  const exampleImage = page.locator('img[alt*="Example PR comment"]').first();
  if (await exampleImage.isVisible()) {
    await exampleImage.scrollIntoViewIfNeeded();
    await page.waitForTimeout(2000);
  }

  // Scroll down to find Quick Start / GitHub Action section
  const quickStartHeading = page.getByRole('heading', { name: /quick start/i }).first();
  if (await quickStartHeading.isVisible()) {
    await quickStartHeading.scrollIntoViewIfNeeded();
    await page.waitForTimeout(1500);
  } else {
    await page.evaluate(() => window.scrollBy({ top: 400, behavior: 'smooth' }));
    await page.waitForTimeout(1500);
  }

  // Scroll to highlight the simplified single workflow file section
  const githubActionHeading = page.getByRole('heading', { name: /github action/i }).first();
  if (await githubActionHeading.isVisible()) {
    await githubActionHeading.scrollIntoViewIfNeeded();
    await page.waitForTimeout(1500);
  } else {
    await page.evaluate(() => window.scrollBy({ top: 400, behavior: 'smooth' }));
    await page.waitForTimeout(1500);
  }

  // Scroll through the workflow YAML snippet to show fewer steps/permissions
  await page.evaluate(() => window.scrollBy({ top: 350, behavior: 'smooth' }));
  await page.waitForTimeout(2000);

  // Continue scrolling to show the config file section
  await page.evaluate(() => window.scrollBy({ top: 350, behavior: 'smooth' }));
  await page.waitForTimeout(1500);

  // Look for config file code block or heading
  const configHeading = page.getByText(/git-glimpse\.config\.ts/i).first();
  if (await configHeading.isVisible()) {
    await configHeading.scrollIntoViewIfNeeded();
    await page.waitForTimeout(2000);
  } else {
    await page.evaluate(() => window.scrollBy({ top: 350, behavior: 'smooth' }));
    await page.waitForTimeout(2000);
  }

  // Scroll a bit more to show the full config snippet with recording/llm options
  await page.evaluate(() => window.scrollBy({ top: 300, behavior: 'smooth' }));
  await page.waitForTimeout(2000);

  // Scroll back to top to end the demo
  await page.evaluate(() => window.scrollTo({ top: 0, behavior: 'smooth' }));
  await page.waitForTimeout(1500);
}

Generated by git-glimpse

git-glimpse logo

@DeDuckProject
Copy link
Copy Markdown
Owner Author

/glimpse

@DeDuckProject DeDuckProject merged commit 020a864 into main Mar 13, 2026
3 checks passed
@DeDuckProject DeDuckProject deleted the some-tweaks branch March 13, 2026 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant