Hide filters on /mcp-server page

[theodorosploumis]

User description

• Update _layouts/default.html

  • Conditionally include filters.html unless page.url contains '/mcp-server'
  • Filters are now hidden on /mcp-server page while remaining on other pages

• Update mcp-server/index.html

  • Remove duplicate h1 title heading (title already set in front matter)

🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

---

PR Type

Enhancement

---

Description

• Hide filters component on /mcp-server page conditionally

• Remove duplicate h1 title from mcp-server index page

---

Diagram Walkthrough

flowchart LR
  A["_layouts/default.html"] -- "conditionally include filters" --> B["filters hidden on /mcp-server"]
  C["mcp-server/index.html"] -- "remove duplicate h1" --> D["clean page structure"]

Loading

File Walkthrough

Relevant files

Enhancement

default.html Conditionally hide filters on mcp-server page                        _layouts/default.html Wrapped filters.html include with conditional logic using unless statement Filters now only render when page URL does not contain '/mcp-server' Maintains filter visibility on all other pages +3/-1      index.html Remove duplicate h1 title heading                                                mcp-server/index.html Removed duplicate h1 heading "Drupal Tools MCP Server" Title is already defined in front matter, eliminating redundancy Cleaned up extra whitespace after removal +1/-2

---

[qodo-code-review]

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢	No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
⚪	🎫 No ticket provided Create ticket/issue
Codebase Duplication Compliance
⚪	Codebase context is not defined Follow the guide to enable codebase context checks.
Custom Compliance
🟢	Generic: Meaningful Naming and Self-Documenting Code Objective: Ensure all identifiers clearly express their purpose and intent, making code self-documenting Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Secure Error Handling Objective: To prevent the leakage of sensitive system information through error messages while providing sufficient detail for internal debugging. Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Secure Logging Practices Objective: To ensure logs are useful for debugging and auditing without exposing sensitive information like PII, PHI, or cardholder data. Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
⚪	Generic: Comprehensive Audit Trails Objective: To create a detailed and reliable record of critical system actions for security analysis and compliance. Status: No auditing impact: The added Liquid conditional for including filters.html does not introduce or modify any critical system actions or logging, so this PR does not affect audit trails. Referred Code {% unless page.url contains '/mcp-server' %} {% include filters.html %} {% endunless %} Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Robust Error Handling and Edge Case Management Objective: Ensure comprehensive error handling that provides meaningful context and graceful degradation Status: No error paths: The change only adds a template conditional to hide filters.html, introducing no new failure points or error handling concerns in the diff. Referred Code {% unless page.url contains '/mcp-server' %} {% include filters.html %} {% endunless %} Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Security-First Input Validation and Data Handling Objective: Ensure all data inputs are validated, sanitized, and handled securely to prevent vulnerabilities Status: Conditional rendering: The new unless page.url contains '/mcp-server' only toggles rendering and does not introduce user input handling or data processing changes visible in this diff. Referred Code {% unless page.url contains '/mcp-server' %} {% include filters.html %} {% endunless %} Learn more about managing compliance generic rules or creating your own custom rules

Compliance status legend

🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

[qodo-code-review]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Use a more specific URL check To make the URL check more specific, replace the Liquid filter contains '/mcp-server' with starts_with '/mcp-server/'. This prevents filters from being unintentionally hidden on pages that are not part of the /mcp-server section but contain the string in their URL. _layouts/default.html [30-32] -{% unless page.url contains '/mcp-server' %} +{% unless page.url starts_with '/mcp-server/' %} {% include filters.html %} {% endunless %} Apply / Chat Suggestion importance[1-10]: 7 __ Why: The suggestion correctly identifies that using contains is too broad and could lead to future bugs. Replacing it with starts_with makes the URL check more specific and robust.	Medium
More