feat: add section_id input to changelog notify workflow#47
Conversation
Define section_id as a required workflow_dispatch input so it is collected on dispatch and forwarded to the backend via SECTION_ID. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Greptile SummaryThis PR wires up the
Confidence Score: 4/5Safe to merge — the change is a one-field addition that completes existing wiring with no logic changes. The change correctly closes the gap between the input definition and the env var that was already in the workflow. The only concern is the ambiguous description showing two path formats, which could cause operators to dispatch with the wrong value.
Important Files Changed
Sequence Diagram%%{init: {'theme': 'neutral'}}%%
sequenceDiagram
actor Operator
participant GHA as GitHub Actions (workflow_dispatch)
participant Script as notify-backend.js
participant API as changelog-api.strettchcloud.com
Operator->>GHA: Dispatch workflow with section_id input
GHA->>Script: Invoke with env vars incl. SECTION_ID
Script->>Script: Read SECTION_ID from env
Script->>API: "POST /api/changelog[/dev] { sectionID, title, summary, ... }"
API-->>Script: HTTP response
Script-->>GHA: Exit 0 / 1
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
sequenceDiagram
actor Operator
participant GHA as GitHub Actions (workflow_dispatch)
participant Script as notify-backend.js
participant API as changelog-api.strettchcloud.com
Operator->>GHA: Dispatch workflow with section_id input
GHA->>Script: Invoke with env vars incl. SECTION_ID
Script->>Script: Read SECTION_ID from env
Script->>API: "POST /api/changelog[/dev] { sectionID, title, summary, ... }"
API-->>Script: HTTP response
Script-->>GHA: Exit 0 / 1
Prompt To Fix All With AIFix the following 1 code review issue. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 1
.github/workflows/changelog-notify.yml:26-28
The description shows two mutually exclusive formats (`"team-creation"` and `"/team-creation"`) without stating which one the backend API expects. If `changelog-api.strettchcloud.com` treats these as distinct values, operators could silently send the wrong format every time they dispatch the workflow. The description should pin the expected format.
```suggestion
section_id:
description: 'Section ID (e.g. "team-creation" — no leading slash)'
required: true
```
Reviews (1): Last reviewed commit: "feat: add section_id input to changelog ..." | Re-trigger Greptile |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
Define section_id as a required workflow_dispatch input so it is collected on dispatch and forwarded to the backend via SECTION_ID.