Skip to content

Add a prompt that summarises an author's reviews - #55

Merged
stphnwlsh merged 1 commit into
mainfrom
feature/author-prompt
Nov 6, 2026
Merged

stphnwlsh merged 1 commit into
mainfrom
feature/author-prompt

Conversation

@stphnwlsh

Copy link
Copy Markdown
Collaborator

Summary

Adds a fourth prompt, summarise_author_reviews, because the other three are all about movies. It embeds the author when it finds one, tells the model to search when it does not, and gets name suggestions from the existing completion handler. The shared message helpers move out of MoviePrompts into PromptContent.

This is stacked on #53 for the icons, and will retarget to main when #53 merges.

Related issues

None.

Type of change

  • Bug fix
  • New feature
  • Breaking change (the generated solution or template parameters change in a way existing users will notice)
  • Refactoring or clean-up
  • Documentation
  • Build, pipeline or dependency update

Shared layers

  • This only touches the Presentation layer of this template.
  • This touches the Application or Infrastructure layers, tests, pipeline or repository setup, so it needs to be carried across all five templates.

Checklist

  • dotnet build and dotnet test pass locally with no warnings.
  • I added or updated tests, and line coverage stays at or above 95%.
  • I updated the README or docs where behaviour changed.
  • If this changes how we do something, I updated the matching skill in .agents/skills.
  • I checked the template still installs and creates a working solution (see CONTRIBUTING.md).

@stphnwlsh stphnwlsh self-assigned this Nov 6, 2026
@github-actions github-actions Bot added the enhancement New feature or request label Nov 6, 2026
Base automatically changed from feature/completions-and-icons to main November 6, 2026 19:01
@github-actions

github-actions Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Build Pipeline

✅ Ready to merge. Every stage passed.

Stage Result Details
Build 🟢 Passed Version 2026.11.06.212-pr-55 · Template package 247 KB
Test 🟢 Passed 291 passed · 0 failed · 0 skipped · Template installs, builds and passes its tests
Coverage 🟢 Passed 100% (1094 of 1094 lines) · Needs 95%
Static Analysis 🟢 Passed No open CodeQL alerts
Dependency Review 🟢 Passed 0 vulnerable packages · 0 licence problems
Container Scan 🟢 Passed Image starts and answers · No open Trivy alerts

Commit fd045e2 · Full run

# Conflicts:
#	tests/Presentation.Tests.Integration/Completions/CompletionTests.cs
@stphnwlsh
stphnwlsh force-pushed the feature/author-prompt branch from 10de470 to fd045e2 Compare November 6, 2026 21:25
@stphnwlsh
stphnwlsh merged commit 6814bfa into main Nov 6, 2026
13 checks passed
@stphnwlsh
stphnwlsh deleted the feature/author-prompt branch November 6, 2026 22:22
stphnwlsh added a commit that referenced this pull request Nov 6, 2026
## Summary

Adds `summarise_movie_reviews`, a prompt that takes a movie id and asks
for what reviewers said: the themes they agree on, where they disagree,
and whether their words match their stars. The `movieId` argument gets
suggestions from the existing completion handler, by title or by the
start of the id.

This is stacked on #55 for `PromptContent`, and will retarget to `main`
when #55 merges. It's most useful once the review text in #56 is in.

## Related issues

None.

## Type of change

- [ ] Bug fix
- [x] New feature
- [ ] Breaking change (the generated solution or template parameters
change in a way existing users will notice)
- [ ] Refactoring or clean-up
- [ ] Documentation
- [ ] Build, pipeline or dependency update

## Shared layers

- [x] This only touches the Presentation layer of this template.
- [ ] This touches the Application or Infrastructure layers, tests,
pipeline or repository setup, so it needs to be carried across all five
templates.

## Checklist

- [x] `dotnet build` and `dotnet test` pass locally with no warnings.
- [ ] I added or updated tests, and line coverage stays at or above 95%.
- [ ] I updated the README or docs where behaviour changed.
- [x] If this changes how we do something, I updated the matching skill
in `.agents/skills`.
- [ ] I checked the template still installs and creates a working
solution (see CONTRIBUTING.md).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant