Skip to content

feat: add section_id input to changelog notify workflow#47

Merged
Valentin-Shyaka merged 1 commit into
mainfrom
add-section-id-changelog-input
Jun 23, 2026
Merged

feat: add section_id input to changelog notify workflow#47
Valentin-Shyaka merged 1 commit into
mainfrom
add-section-id-changelog-input

Conversation

@Valentin-Shyaka

Copy link
Copy Markdown
Contributor

Define section_id as a required workflow_dispatch input so it is collected on dispatch and forwarded to the backend via SECTION_ID.

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-apps

greptile-apps Bot commented Jun 23, 2026

Copy link
Copy Markdown

Greptile Summary

This PR wires up the section_id workflow dispatch input that was already referenced (but never collected) in the SECTION_ID environment variable passed to notify-backend.js, making the field actually usable at dispatch time.

  • Adds section_id as a required: true workflow_dispatch input; the SECTION_ID env var and its use as sectionID in the JSON payload already existed in the script unchanged.
  • The input description lists two formats ("team-creation" and "/team-creation") without specifying which the backend expects, which could lead to inconsistent values being dispatched.

Confidence Score: 4/5

Safe 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.

.github/workflows/changelog-notify.yml — the section_id description should clarify the exact expected format.

Important Files Changed

Filename Overview
.github/workflows/changelog-notify.yml Adds the missing section_id workflow_dispatch input (required, no default) that was already wired into the job's env as SECTION_ID — completing the previously broken data flow to the backend script.

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
Loading
%%{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
Loading
Prompt To Fix All With AI
Fix 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

Comment thread .github/workflows/changelog-notify.yml
@mintlify

mintlify Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
strettch-cloud 🟢 Ready View Preview Jun 23, 2026, 8:19 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@Valentin-Shyaka Valentin-Shyaka merged commit aa4f970 into main Jun 23, 2026
5 checks passed
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