[PLA-2198] Add canary-matrix v1020

[leonardocustodio]

PR Type

Enhancement

---

Description

• Added support for v1020 in the canary matrix.

• Updated canaryLatest and getLatestSpecVersion to include v1020.

• Adjusted export order in canary.dart for consistency.

• Introduced new file v1020.dart for canary matrix specifications.

---

Changes walkthrough 📝

Relevant files

Enhancement

canary.dart Add and reorder exports in `canary.dart`                                  lib/consts/matrix/canary/canary.dart Added export for v1020.dart. Reordered existing exports for better organization. +7/-6      matrix.dart Update matrix logic to include `v1020`                                      lib/consts/matrix/matrix.dart Updated canaryLatest to return v1020. Added case for v1020 in canarySpec. +3/-1      general.dart Update `getLatestSpecVersion` for `v1020`                                lib/handler/general.dart Updated getLatestSpecVersion to return v1020 for canary networks. +1/-1      v1020.dart Add new canary matrix version file `v1020`                              lib/consts/matrix/canary/v1020.dart Added new file defining specifications for v1020. +2/-0

---

Need help?

Type /help how to ... in the comments thread for any questions about PR-Agent usage.

Check out the documentation for more information.

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Possible Issue The canarySpec function now includes a case for 1020, but the default case still returns canaryLatest(). Ensure that this behavior is intentional and aligns with the expected logic for unsupported versions. case 1020: return c.v1020; default: return canaryLatest(); Logic Update The getLatestSpecVersion function now returns 1020 for the canary network. Confirm that this change does not conflict with other parts of the system relying on the previous value 1013. return 1020; }

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Validate network parameter values Validate the network parameter in getLatestSpecVersion to ensure it only accepts expected values, preventing potential misuse or errors. lib/handler/general.dart [18-20] -if (network == 'canary' || network == 'canary-matrixchain') { +if (['canary', 'canary-matrixchain'].contains(network)) { return 1020; } Suggestion importance[1-10]: 8 __ Why: Validating the network parameter ensures that only expected values are processed, reducing the risk of misuse or errors. This is a significant improvement in terms of input validation and code reliability.	Medium
	Add fallback for invalid spec versions Ensure that canaryLatest and canarySpec functions handle unexpected or invalid spec versions gracefully to avoid potential runtime errors. lib/consts/matrix/matrix.dart [4-7] String canaryLatest() { - return c.v1020; + return c.v1020 ?? 'default_value'; } ... case 1020: - return c.v1020; + return c.v1020 ?? 'default_value'; Suggestion importance[1-10]: 7 __ Why: Adding a fallback for invalid spec versions improves the robustness of the canaryLatest and canarySpec functions, preventing potential runtime errors. However, the suggestion assumes the use of a default value ('default_value') without clarifying its appropriateness or context, which slightly reduces its impact.	Medium