Skip to content

Fix member-matching and clean up generated_content schema#134

Merged
denbec merged 2 commits intomainfrom
denbec/check-podcast-fields
Feb 10, 2026
Merged

Fix member-matching and clean up generated_content schema#134
denbec merged 2 commits intomainfrom
denbec/check-podcast-fields

Conversation

@denbec
Copy link
Collaborator

@denbec denbec commented Feb 10, 2026

Summary

  • Fixed member-matching hook regex to prevent matching across newlines
  • Added null safety check for member names
  • Filter out archived members from matching
  • Remove unused fields from podcast_generated_content collection (llm_model, prompt_version, edited_text, approved_by)

Test plan

  • Verify member-matching correctly identifies speakers from transcript with timestamp format
  • Confirm archived members are excluded from auto-assignment
  • Ensure content generation and heise integration hooks work without removed fields

denbec and others added 2 commits February 10, 2026 16:57
- Use [ \t]+ instead of \s+ in timestamp regex to prevent matching
  across newlines, which caused entire paragraphs to be extracted
  as speaker names
- Add null guard in normalize() to handle members with null names
- Filter out archived members from matching query

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove llm_model and prompt_version from all createOne calls. These
fields were removed from the collection schema and no longer needed.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Feb 10, 2026

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

Project Deployment Actions Updated (UTC)
programmierbar-website Ready Ready Preview, Comment Feb 10, 2026 4:05pm

Request Review

@denbec denbec merged commit 1d9f41c into main Feb 10, 2026
3 checks passed
@denbec denbec deleted the denbec/check-podcast-fields branch February 13, 2026 12:41
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.

1 participant