RDKEMW-26909 : Align CI workflows with heatmap standard

[swethasukumarr]

No description provided.

[Copilot]

🟡 Changes recommended

The coding guidelines reference an unavailable authoritative specification and incorrectly identify the active CI mock-version pin.

4 open findings

• Referenced authoritative API specification is missing · New
• Incorrectly claims standard string types cannot throw · New
• CI pin update rule references the wrong workflow file · New
• Literal escape sequences prevent Markdown formatting · New

What changed in this PR

Aligns CI workflows with the heatmap standard by extracting reusable test workflows and updating automation conventions.

Changes:

• Splits L1 and L2 tests into reusable workflows.
• Updates release, synchronization, coverage, and transport configuration.
• Removes the temporary TextToSpeech test exclusion and expands coding guidance.

File	Description
README.md	Removes the temporary exclusion notice.
docs/​ci/​text-to-speech-component-test-exclusion.md	Deletes obsolete exclusion documentation.
.transport.version	Updates the transport dependency.
.github/​workflows/​tests-trigger.yml	Delegates tests to reusable L1/L2 workflows.
.github/​workflows/​sync-develop-to-main.yml	Renames the automation token.
.github/​workflows/​release.yml	Aligns release branches and token usage.
.github/​workflows/​L2-tests.yml	Adds reusable component and API testing.
.github/​workflows/​L1-tests.yml	Adds reusable unit testing and coverage.
.github/​scripts/​compare_coverage_test.py	Updates workflow terminology.
.github/​instructions/​coding-guidelines.instructions.md	Expands repository development guidance.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

🟡 Changes recommended

The coding guidance introduces an unavailable authoritative specification and contains inaccurate verification notes.

1 open finding

• Do not make an untracked draft the authoritative API specification · New

4 resolved since last review

• Literal escape sequences prevent Markdown formatting
• CI pin update rule references the wrong workflow file
• Incorrectly claims standard string types cannot throw
• Referenced authoritative API specification is missing

Previously missed (2)

In code that hasn't changed since last review

Remove stale VERIFY marker for existing repository files

.github/​instructions/​coding-guidelines.instructions.md:516

This verification item is already resolvable from the repository: the script maps to .github/mock-firebolt/config--legacy.json and docs/openrpc/the-spec/firebolt-open-rpc--legacy.json, and both files exist; the app fixture is deliberately ignored. Leaving this marked [VERIFY] contradicts the marker's definition and gives readers stale guidance.

Enable pipefail in the Docker subprocess

.github/​instructions/​coding-guidelines.instructions.md:673

lint.sh itself already starts with set -euo pipefail; the missing pipefail is specifically inside its Docker bash -lc "set -e && ... | ..." subprocess. Describing the script as having no pipefail obscures where the fix must be applied.

🧠 Review effort: Balanced

[Copilot]

🟡 Changes recommended

Legacy CI now runs TextToSpeech event tests against incompatible legacy event names.

1 open finding

• Legacy fixture event casing breaks TextToSpeech tests · New

1 resolved since last review

• Do not make an untracked draft the authoritative API specification

🧠 Review effort: Balanced