Fix TeamDynamix auth scripting config

[shawn149]

📋 Summary

This PR fixes a bug where the TeamDynamix plugin never sent an Authorization header, so every request to a customer's portal came back 401.

The plugin sets preRequestScript to do the BEID / Web Services Key token exchange, but not showScripting, which defaults to false. When that flag is falsy the WebAPI base plugin discards the script before running it, with no error and no warning, so the plugin looks correctly wired while sending unauthenticated requests. In the manual WebAPI data source showScripting is the "Pre-request script" toggle a user ticks, and a declarative plugin has no one to tick it, so it has to set the flag itself. I've set it to true and bumped the plugin to 1.0.2.

I've also added the flag to the build-plugin skill's pre-request section, which covered scriptingVariables and preRequestScript but never mentioned showScripting. Anyone following that guidance ends up with the same silent failure.

To verify, run any data stream against a configured TeamDynamix data source (for example Tickets): before this change every request comes back 401, after it the token exchange runs and the stream returns rows. I haven't been able to test against a live portal, so this is verified against how the platform handles showScripting rather than end to end, and it's worth a reviewer with portal access confirming before it goes out.

---

🧩 Plugin details

• Plugin name: TeamDynamix
• Type of change:
  • Bug fix
  • New datastream
  • Enhancement to existing datastream
  • Performance improvement
  • Documentation / metadata / logo
  • Other (please describe):

---

⚠️ Breaking changes

Does this PR introduce any breaking changes?

• No
• Yes (please describe):

---

📚 Documentation

• Documentation updated
• No documentation changes needed

---

✅ Checklist

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

Summary by CodeRabbit

• New Features

  • Enabled pre-request scripting support for the TeamDynamix WebAPI integration.
  • Added the required scripting visibility setting to ensure configured pre-request scripts run correctly.

• Documentation

  • Updated configuration guidance to explain the required scripting setting and behavior when it is disabled.
  • Clarified that static header diagnostics may appear while requests remain unauthenticated.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Pro

Run ID: dc0fcac1-bd42-4865-86e1-ecce886ba20a

📥 Commits

Reviewing files that changed from the base of the PR and between 84b5d1a and 9351492.

📒 Files selected for processing (2)

• .claude/skills/build-plugin/references/metadata.md
• plugins/TeamDynamix/v1/metadata.json

---

📝 Walkthrough

Walkthrough

The TeamDynamix plugin now enables WebAPI scripting and reports version 1.0.2. The plugin build documentation explains that showScripting: true is required for pre-request scripts.

Changes

TeamDynamix scripting configuration

Layer / File(s)	Summary
Enable WebAPI scripting plugins/TeamDynamix/v1/metadata.json, .claude/skills/build-plugin/references/metadata.md	The metadata enables showScripting for the WebAPI base and increments the plugin version to 1.0.2. The reference documents the required flag and explains that falsy values skip the pre-request script and can send unauthenticated requests.

Mergeability Score: ⚪ Minimal · up to 93514

This localized configuration and documentation update enables the TeamDynamix authorization script without introducing a supported merge-blocking concern; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly identifies the TeamDynamix authentication scripting configuration fix, which is the main change.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

🧩 Plugin PR Summary

📦 Modified Plugins

• plugins/TeamDynamix/v1

📋 Results

Step	Status
Validation	✅ Passed
Deployment	🚀 Deployed

🔍 Validation Details

✅ teamdynamix

{
  "valid": true,
  "pluginName": "teamdynamix",
  "pluginType": "hybrid",
  "summary": {
    "Data Streams": 13,
    "Import Definitions": 1,
    "Correlation Rules": 0,
    "UI Configuration": true,
    "Has Icon": true,
    "Has Default Content": true,
    "Config Validation": true,
    "Custom Types": true
  }
}