Skip to content

Add recipe editing to the web client#127

Open
jschoedl wants to merge 4 commits into
mainfrom
feat/recipe-editing-ct
Open

Add recipe editing to the web client#127
jschoedl wants to merge 4 commits into
mainfrom
feat/recipe-editing-ct

Conversation

@jschoedl

@jschoedl jschoedl commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds an edit page for recipes, reachable from the recipe view for both generated (unsaved) recipes and saved library recipes. Editing a library recipe saves via PUT /recipes/{id}; editing a generated recipe asks whether to also store it in the library. Unsaved edits are guarded by a discard prompt on navigation and on tab close.

Also extracts the auto-growing AutoTextarea shared by the editor and the existing "get help" input, and adds pact + Spring API contract coverage for PUT /recipes/{id}.

Type of change

  • Bug fix
  • New feature
  • Refactor / cleanup
  • Infrastructure / CI
  • Documentation

API changes

  • This PR does not affect the API
  • This PR changes the API → api/openapi.yaml updated and api/scripts/gen-all.sh re-run

Definition of Done

  • CI passes
  • Pre-commit hooks pass locally
  • Relevant tests added or updated

🤖 Generated with Claude Code

jschoedl and others added 3 commits July 9, 2026 17:47
Verify py-recipe-service and py-help-service responses conform to the
internal OpenAPI contract, run in-process over ASGI with the LLM stubbed.
Both service CI workflows now also trigger on api/openapi-internal.yaml.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Consumer-driven Pact over the public REST boundary: the web-client
consumer test generates web-client/pacts/, which the spring-api provider
test verifies against the real app (JWT auth real, Python GenAI clients
mocked). Covers every route the client uses, happy and error paths.
The committed pact is the filesystem handoff; spring-api CI triggers on it.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Editable pages for both generated (unsaved) and library recipes, backed by
PUT /recipes/{id} pact and Spring API contract coverage.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

This comment was marked as low quality.

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

🚀 Frontend preview ↗  ·  commit 8d71f629b1a5a61aff5f1462cc21d6d91c815b9b

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jschoedl jschoedl requested review from imol-ai and paulwiese July 9, 2026 16:22
@jschoedl jschoedl linked an issue Jul 9, 2026 that may be closed by this pull request

@paulwiese paulwiese left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

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.

Manually edit recipe

2 participants