Add paging support to Rootly- incidents datastream

[noorulkhan-n]

📋 Summary

• Added offset-based paging support for the incidents datastream.
• Configured page size and offset handling to retrieve all records across multiple pages.
• Ensures incident imports continue correctly when API responses exceed a single page.

🧩 Plugin details

• Plugin name:

• Type of change:

  • Bug fix
  • New datastream
  • Enhancement to existing datastream
  • Performance improvement
  • Documentation / metadata / logo
  • Other (please describe)

⚠️ Breaking changes

• No
• Yes (please describe)

📚 Documentation

• No documentation changes needed
• Documentation updated

✅ Checklist

• No secrets or credentials included
• Plugin, datastream and UI naming follow SquaredUp guidelines
• I agree to the Code of Conduct

[clarkd]

@claude review

[clarkd]

Hi @noorulkhan-n - please update the PR description to use one of the templates in the current PR description.

[noorulkhan-n]

@clarkd updated the description now

[clarkd]

@claude review once

[clarkd]

Version bump required, but the rest of the change looks good

[noorulkhan-n]

@clarkd version incremented

[github-actions]

🧩 Plugin PR Summary

📦 Modified Plugins

• plugins/Rootly/v1

📋 Results

Step	Status
Validation	✅ Passed
Deployment	🚀 Deployed

🔍 Validation Details

✅ rootly

{
  "valid": true,
  "pluginName": "rootly",
  "pluginType": "cloud",
  "summary": {
    "Data Streams": 1,
    "Import Definitions": 0,
    "UI Configuration": true,
    "Has Icon": true,
    "Has Default Content": false,
    "Config Validation": false,
    "Custom Types": false
  }
}

[clarkd]

@claude review once

[claude]

All prior feedback addressed (version bump, tags added) — straightforward paging config enhancement.

Extended reasoning...

Overview

This PR adds offset-based paging configuration to the Rootly incidents data stream (page size 200, using the WebAPI paging block with mode: offset and mode: page), bumps the plugin version to 1.0.3, removes the unrecognized modules: [] key, and adds a top-level tags: ["Incidents"] field on the data stream. Only two files are touched: plugins/Rootly/v1/dataStreams/incidents.json and plugins/Rootly/v1/metadata.json.

Security risks

None. This is a declarative JSON config change to an existing data stream — no script execution paths, no auth changes, no new secrets, no URL changes. The Authorization: Bearer {{apiKey}} header is untouched.

Level of scrutiny

Low. This is a small, mechanical config-only change following the established WebAPI paging pattern used elsewhere in the repo. The PR's footprint is two JSON files within one plugin's v1 directory, and the changes are easy to reason about line-by-line from the diff.

Other factors

The author has responded to both of my prior reviews — the version bump (1.0.2 → 1.0.3) was added after my first review, and the tags field was added after my second review. The CI plugin validation also passed and reports a successful deployment. The only outstanding item is a pre-existing nit about the missing docs/README.md and documentation link, which I've explicitly noted is not blocking. Nothing here warrants holding up the merge.