Skip to content

Remove hidden README and docs navigation text - #53816

Merged
pelikhan merged 4 commits into
mainfrom
copilot/deep-report-investigate-hidden-text-cloaking
Aug 18, 2026
Merged

Remove hidden README and docs navigation text#53816
pelikhan merged 4 commits into
mainfrom
copilot/deep-report-investigate-hidden-text-cloaking

Conversation

Copilot AI commented Aug 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

The GEO audit flagged hidden text in the README and docs site as a cloaking risk. This change removes non-visible instructional content and avoids shipping hidden navigation text in the closed responsive menu.

  • README

    • Removed the leading agent-only HTML comment.
    • Removed the collapsed setup guidance block.

  • Docs header

    • Keeps desktop navigation server-rendered.
    • Populates responsive dropdown links only when the hamburger menu opens.
    • Clears dropdown contents when the menu closes.
populateDropdown();
tabletDropdown.hidden = false;
// ...
tabletDropdown.replaceChildren();
  • Regression coverage
    • Added a Playwright guard for hidden README setup guidance.
    • Added a responsive-header check that the closed dropdown contains no hidden link text.

Copilot AI linked an issue Aug 18, 2026 that may be closed by this pull request
[deep-report] Investigate hidden-text cloaking flag on docs site and README #53792
Closed
Copilot AI and others added 2 commits August 18, 2026 20:27
@pelikhan
fix: remove hidden README and docs nav text
030ff4b 
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan
test: tighten hidden text guard viewport
37e30e7 
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Investigate hidden text cloaking flag on docs site and README Remove hidden README and docs navigation text Aug 18, 2026
Copilot AI requested a review from pelikhan August 18, 2026 20:32
@pelikhan

pelikhan commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

@copilot keep the cloaked comment as a details section

@pelikhan
fix: keep agent quick links visible in README
8b7b37a 
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Aug 18, 2026

Copy link
Copy Markdown
Contributor Author

@copilot keep the cloaked comment as a details section

Addressed in 8b7b37ae54 by restoring the former agent-only README comment as a visible <details> section while keeping the setup guidance removed.

@pelikhan
pelikhan marked this pull request as ready for review August 18, 2026 21:20
Copilot AI balanced review requested due to automatic review settings August 18, 2026 21:20
@pelikhan
pelikhan merged commit 4845f00 into main Aug 18, 2026
@pelikhan
pelikhan deleted the copilot/deep-report-investigate-hidden-text-cloaking branch August 18, 2026 21:20
Copilot AI reviewed Aug 18, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates README and responsive docs navigation to reduce hidden text flagged by the GEO audit.

Changes:

  • Reworks README agent guidance and removes setup instructions.
  • Dynamically populates and clears responsive navigation links.
  • Adds Playwright regression coverage.
Show a summary per file
File Description
README.md Revises agent guidance visibility.
docs/src/components/CustomHeader.astro Dynamically manages responsive links.
docs/tests/no-hidden-cloaking.spec.ts Adds hidden-text regression tests.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 3/3 changed files
  • Comments generated: 3
  • Review effort level: Balanced

Comment thread README.md
Comment on lines +3 to +4
<details>
<summary>Agent quick links</summary>
Comment thread docs/src/components/CustomHeader.astro
Comment on lines 18 to +19
<nav class="custom-header-links" aria-label="Primary navigation">
<a href={`${base}setup/quick-start/`} class="header-link">Quick Start</a>
<a href={`${base}setup/creating-workflows/`} class="header-link">Create</a>
<a href={`${base}index.html#gallery`} class="header-link">Examples</a>
<a href={`${base}introduction/overview/`} class="header-link">Docs</a>
<a href={`${base}reference/faq/`} class="header-link">FAQ</a>
<a href={`${base}blog/`} class="header-link">Blog</a>
<a href={`${base}blog/2026-01-12-welcome-to-pelis-agent-factory/`} class="header-link">Peli's Agent Factory</a>
{navItems.map((item) => <a href={item.href} class="header-link">{item.label}</a>)}
Comment thread docs/src/components/CustomHeader.astro
Comment on lines +153 to +156
...navItems.map((item) => {
const link = document.createElement('a');
link.href = item.href;
link.className = 'dropdown-link';
@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.87.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort, defaults to Balanced
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@pelikhan pelikhan Awaiting requested review from pelikhan

Assignees

@pelikhan pelikhan

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[deep-report] Investigate hidden-text cloaking flag on docs site and README

3 participants

@pelikhan