Skip to content
This repository was archived by the owner on Jul 6, 2026. It is now read-only.

Update blog-post-reviewer agent documentation#2784

Closed
MaxGhenis wants to merge 3 commits into
mainfrom
update-blog-reviewer-docs
Closed

Update blog-post-reviewer agent documentation#2784
MaxGhenis wants to merge 3 commits into
mainfrom
update-blog-reviewer-docs

Conversation

@MaxGhenis

Copy link
Copy Markdown
Contributor

Summary

Updates the blog-post-reviewer agent with documentation for the external_url pattern used by interactive calculators and research tools.

Changes

  • Added external_url as optional field in posts.json documentation
  • Added guidance on avoiding redundant location markers (e.g., "Two-child limit calculator" not "UK two-child limit calculator" on UK-only sites)
  • New section on "Interactive Tools and Calculators" with best practices:
    • Concise naming conventions (prefer "calculator" over "policy comparison")
    • Action-oriented descriptions
    • Using "interactives" tag
    • Pattern of using dummy filename with explanatory comment

Example

Codifies the pattern established in #2783 (two-child limit calculator).

🤖 Generated with Claude Code

MaxGhenis and others added 2 commits October 28, 2025 21:26
Added documentation for external_url pattern used by interactive calculators and tools:
- Optional external_url field for linking to app routes instead of markdown posts
- Guidance on concise naming (prefer "calculator" over "comparison")
- Best practice to avoid redundant location markers on country-specific sites
- Example showing the two-child limit calculator pattern

This codifies the pattern we established in PR #2783.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings October 29, 2025 01:27
@vercel

vercel Bot commented Oct 29, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
policyengine-app Ready Ready Preview Comment Oct 29, 2025 0:45am

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

This PR updates the blog-post-reviewer agent documentation to formalize the pattern for interactive tools and calculators. The change reflects a documentation update (patch bump) rather than a new feature.

  • Added comprehensive guidelines for interactive tools pattern using external_url field
  • Documented naming conventions and best practices for calculator-type research tiles
  • Included a concrete JSON example demonstrating the interactive tools pattern

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
changelog_entry.yaml Changed from minor to patch bump and updated description to reflect documentation change
.claude/agents/blog-post-reviewer.md Added new section documenting interactive tools pattern with guidelines and example

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MaxGhenis

Copy link
Copy Markdown
Contributor Author

Thank you for this contribution. Closing as part of the policyengine-app (v1) wind-down: policyengine.org is now served by policyengine-app-v2, and this repository is being archived, so changes merged here would not reach the live site.

If this is still relevant, please consider re-opening it against policyengine-app-v2 — we'd welcome it there. Apologies that this sat open through the migration.

@MaxGhenis MaxGhenis closed this Jul 6, 2026
@github-project-automation github-project-automation Bot moved this from Todo to Closed in policyengine-app Jul 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: Closed

Development

Successfully merging this pull request may close these issues.

2 participants