Skip to content

fix: improve trigger error handling, markdown rendering, and release workflow#633

Merged
yaojin3616 merged 2 commits into
mainfrom
feature/update-release-workflow-triggers
May 29, 2026
Merged

fix: improve trigger error handling, markdown rendering, and release workflow#633
yaojin3616 merged 2 commits into
mainfrom
feature/update-release-workflow-triggers

Conversation

@yaojin3616
Copy link
Copy Markdown
Collaborator

Changes

Backend

  • Trigger error handling (): Mark trigger executions as failed when agent is not found/expired, has no LLM model, or model is unavailable — instead of silently skipping
  • Session safety (): Add safety checks before expunging execution/trigger objects from DB session to prevent errors

Frontend

  • Markdown rendering (): Fix bold/italic regex patterns to avoid matching underscores inside words (e.g., )

CI/CD

  • Release workflow (): Update GitHub Models draft step to use

yaojin added 2 commits May 29, 2026 23:57
…workflow

- Mark trigger executions as failed when agent is not found/expired, has no LLM model, or model is unavailable instead of silently skipping
- Add safety checks before expunging execution/trigger objects from DB session
- Fix markdown bold/italic regex to avoid matching underscores inside words
- Update GitHub Models draft step to use openai/gpt-5.4
@yaojin3616 yaojin3616 force-pushed the feature/update-release-workflow-triggers branch from a523c28 to 387536a Compare May 29, 2026 15:57
@yaojin3616 yaojin3616 merged commit 0b1a22d into main May 29, 2026
1 check was pending
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a523c286c9

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

shell: bash
env:
MODELS_TOKEN: ${{ secrets.MODELS_TOKEN }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Grant models permission before using GITHUB_TOKEN

When use_ai_notes is true, this step now authenticates GitHub Models with the workflow GITHUB_TOKEN, but the workflow permissions block only grants contents and pull-requests; GitHub Actions sets omitted permissions to none, and the GitHub Models quickstart/API docs require models: read for workflows calling the inference endpoint. As a result, the curl request will return an authorization error and fail every AI-notes release run unless models: read is added.

Useful? React with 👍 / 👎.

@yaojin3616 yaojin3616 deleted the feature/update-release-workflow-triggers branch May 29, 2026 15:58
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