Skip to content

chore: consolidate workshop writing agents - #187

Merged
GeekTrainer merged 1 commit into
mainfrom
geektrainer/consolidate-writing-agents
Sep 18, 2026
Merged

GeekTrainer merged 1 commit into
mainfrom
geektrainer/consolidate-writing-agents

Conversation

@GeekTrainer

Copy link
Copy Markdown
Collaborator

Summary

Replace three generic writing, accessibility, and content-review agents with one repository-focused Tech Writer agent to reduce overlapping guidance and keep authoring aligned with Copilot Workshops conventions. The new agent emphasizes verified technical facts, developer-focused numbered actions, aligned objectives and summaries, and existing repository accessibility and validation guidance.

Type of change

  • Workshop content (lesson Markdown, images)
  • Site shell (website/ Astro + Starlight wrapper)
  • Copilot configuration (.github/copilot-instructions.md, instructions, agents, skills)
  • Repo housekeeping (CI, dependabot, README, license)
  • Other:

Verification

  • Type checks and a clean site build pass. User confirmed the repository verification wrapper passed in this worktree; not rerun by the agent.
  • Affected pages and translations match their source and intended routes. N/A: no published content or routes changed.
  • Offline internal link checks pass. Included in the user-confirmed verification wrapper.
  • Changed external links have been checked separately. N/A: no external links added or updated.

Also checked for stale references to the removed agents outside their definitions; none found. git diff --check passed.

Screenshots

N/A: agent configuration only.

Notes for reviewers

The standalone Markdown Accessibility Assistant, SE: Tech Writer, and technical-content-evaluator agents are removed. The replacement delegates formatting and accessibility conventions to existing repository instructions instead of duplicating them, and avoids generic grading formulas and unnecessary documentation templates. No workshop lessons or site code change.

Replace three generic writing and review agents with a repository-focused Tech Writer agent.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 18, 2026 17:23
@GeekTrainer
GeekTrainer merged commit 949044f into main Sep 18, 2026
7 checks passed

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.

Copilot review overview

🟡 Changes recommended

Address the two moderate findings in the consolidated Tech Writer agent.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 Medium severity

Open (1)
What changed in this PR

Consolidates three writing and review agents into one repository-focused Tech Writer agent.

Changes:

  • Adds unified authoring and validation guidance.
  • Removes three redundant agents.
  • Reuses existing repository accessibility and localization conventions.
File Summary
.github/​agents/​technical-content-evaluator.agent.md Removes the legacy content evaluator.
.github/​agents/​tech-writer.agent.md Adds the consolidated agent; needs support for lesson, exercise, and module headings, plus localization-aware structural changes.
.github/​agents/​se-technical-writer.agent.md Removes the generic technical writer.
.github/​agents/​markdown-accessibility-assistant.agent.md Removes the standalone accessibility assistant.

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


- Do not invent product behavior, UI labels, commands, file paths, or technical results. Verify them in the repository, the Tailspin Toys application, or authoritative documentation.
- Do not install dependencies, create commits, push branches, or open pull requests unless the user explicitly requests and approves that work.
- Do not update translations unless the requested scope includes them. Identify affected localized content when relevant.
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.

2 participants