Skip to content

test: verify /glimpse uses PR branch code#24

Closed
DeDuckProject wants to merge 1 commit intomainfrom
feat/verify-branch-code-runs
Closed

test: verify /glimpse uses PR branch code#24
DeDuckProject wants to merge 1 commit intomainfrom
feat/verify-branch-code-runs

Conversation

@DeDuckProject
Copy link
Copy Markdown
Owner

Verification PR for the fix in #22.

This branch adds · branch-code-test to the PR comment footer — a change that only exists on this branch, not on main.

How to verify: Comment /glimpse on this PR. If the posted comment ends with · branch-code-test, the fix works and /glimpse is correctly using the PR branch's action code.

This PR will be closed/reverted after verification.

🤖 Generated with Claude Code

Temporary marker in PR comment footer. If the comment ends with
"· branch-code-test", the fix works and branch code is being used.
Remove after verification.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@DeDuckProject
Copy link
Copy Markdown
Owner Author

/glimpse

@github-actions
Copy link
Copy Markdown

git-glimpse logo UI Demo Preview

Changes detected in: packages/action/dist/index.js, packages/action/dist/index.js.map, packages/core/src/publisher/github-comment.ts

What changed: The footer attribution text in generated PR/issue comments now includes a branch identifier suffix ('· branch-code-test') appended after the git-glimpse credit line, making it easier to identify which branch or version of the action produced the comment.

Demo

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

export async function demo(page: Page): Promise<void> {
  // Navigate to the home page
  await page.goto('http://localhost:3000');
  await page.waitForLoadState('networkidle');
  await page.waitForTimeout(1500);

  // Scroll to the bottom of the page to find the generated comment footer
  await page.evaluate(() => window.scrollTo(0, document.body.scrollHeight));
  await page.waitForTimeout(1500);

  // Try to find text containing 'Generated by git-glimpse' and scroll it into view
  const footerLocator = page.locator('text=Generated by git-glimpse').first();
  const footerExists = await footerLocator.count() > 0;

  if (footerExists) {
    await footerLocator.scrollIntoViewIfNeeded();
    await page.waitForTimeout(2000);

    // Hover over the footer text to highlight it
    await footerLocator.hover();
    await page.waitForTimeout(2000);
  } else {
    // Try to find any element with 'branch-code-test' in it
    const branchLocator = page.locator('text=branch-code-test').first();
    const branchExists = await branchLocator.count() > 0;

    if (branchExists) {
      await branchLocator.scrollIntoViewIfNeeded();
      await page.waitForTimeout(2000);
      await branchLocator.hover();
      await page.waitForTimeout(2000);
    } else {
      // Fall back: scroll through the page slowly to demonstrate the UI
      const scrollHeight = await page.evaluate(() => document.body.scrollHeight);
      const viewportHeight = 720;
      const steps = 5;

      for (let i = 1; i <= steps; i++) {
        await page.evaluate((y) => window.scrollTo(0, y), (scrollHeight / steps) * i);
        await page.waitForTimeout(800);
      }

      await page.waitForTimeout(1500);
    }
  }

  // Scroll back up slightly to show context
  await page.evaluate(() => window.scrollBy(0, -200));
  await page.waitForTimeout(1500);

  // Final pause to let the recording capture the updated footer clearly
  await page.waitForTimeout(2000);
}

Generated by git-glimpse · branch-code-test

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