| name | develop-issue |
| description | Implement a GitHub issue end-to-end in a Wazuh Dashboard repo — plan, code following repo conventions, add colocated tests, validate with check-standards, add the CHANGELOG entry, and deliver a filled PR-template body (leaving Results and Evidence/screenshot to the developer) WITHOUT opening the PR. Use when the user provides an issue to develop, implement, or work on. |
Develop an issue (issue → code → tests → delivery)
Entry point for feature/bug work. Takes an issue, produces the change plus a
ready-to-paste PR delivery, and hands off to the developer for the screenshot and
the final PR creation. Chains the check-standards and create-pr skills.
Workflow
Copy this checklist and track progress:
- [ ] 1. Plan from the issue (goal, acceptance criteria, affected layer)
- [ ] 2. Implement, focused on the issue, respecting conventions
- [ ] 3. Add/update colocated tests and run them
- [ ] 4. Validate with check-standards; fix failures
- [ ] 5. Add the CHANGELOG entry
- [ ] 6. Deliver via create-pr (prepare mode) — do NOT open the PR
1. Plan
Issues are normally shared as a URL and may live in a different repo. Read it
first and classify the source:
gh issue view <issue-url>
- Internal — URL contains
internal-devel-request (e.g.
wazuh/internal-devel-requests): the issue link is not exposed in the PR
(## Description gets no closing reference) and there is no CHANGELOG
entry.
- Public — any other repo (e.g.
wazuh/wazuh-dashboard-alerting): link it in
the PR and add a CHANGELOG entry pointing to the issue.
Restate the issue's goal and acceptance criteria in your own words. Identify the
affected layer(s) — public (browser/React), server (Node/routes/services).
Ask the user only if genuinely blocked; otherwise proceed with reasonable
defaults.
2. Implement
Keep the change scoped to the issue. Respect the architecture and conventions in
CLAUDE.md:
- Never import
server/ from public/ or vice versa; shared code goes in a
shared module (this plugin has no common/ dir — follow the surrounding
structure). Cross-plugin: public → other/public, server → other/server,
preferably via setup()/start() contracts.
- This is a fork of
opensearch-project/alerting-dashboards-plugin: match the
upstream filename convention (PascalCase for components), English
everywhere. It is primarily JavaScript (Redux Toolkit, Formik, react-vis);
the OpenSearch license header is NOT enforced here, so do not add one. On
upstream syncs, Wazuh content wins.
3. Tests (colocated)
Add or update unit tests as *.test.js / *.test.tsx, typically inside
__tests__/ folders next to the changed source (snapshots under
__snapshots__/). New functionality must include testing.
repo-specific (wazuh-dashboard-alerting): this plugin's scripts run from
inside the wazuh-dashboard checkout at plugins/<this-plugin> (they
reference ../../node_modules/.bin/jest). Bootstrap the platform once
(yarn osd bootstrap from the wazuh-dashboard root), then run
yarn test:jest here (not yarn test); refresh snapshots with
yarn test:jest:update-snapshots.
4. Validate
Run the check-standards skill (prettier + eslint + tests over the diff; this
JS-first repo has no typecheck step). Fix everything it reports before delivering.
5. CHANGELOG
For public issues, add an entry under the upcoming version in
CHANGELOG.md (Added / Changed / Fixed /
Removed), with the link pointing to the issue (not the PR). Skip the entry
for internal-devel-requests issues, and for tooling/docs/test-only changes
(use the no-changelog label on the PR).
6. Deliver (do NOT open the PR)
Invoke create-pr in its default prepare-and-hand-off mode. Output:
- The filled PR-template body, with the
## Description completed (public
issue → Closes #<n> / issue URL; internal-devel-requests → no closing
reference), the ### Review Checklist boxes that genuinely apply checked,
and a screenshot/video reminder under ### Results and Evidence for any
UI change.
- The pre-flight report (branch, suggested base, DCO status, check-standards
result, CHANGELOG status, and the
gh pr create command to run when ready).
End by reminding the developer to sign commits with --signoff and to add the
screenshot before opening the PR.
