Skip to content

Add plan diff information in the comment - #291

Merged
refeed merged 3 commits into
mainfrom
feat/plan-diff-in-comment
Sep 4, 2026
Merged

refeed merged 3 commits into
mainfrom
feat/plan-diff-in-comment

Show attribute-level changes in the plan diff (#292)

e4544d0
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis cancelled Sep 4, 2026 in 28s

SonarQube Cloud analysis failed

❌ The last analysis has failed.

See analysis details on SonarQube Cloud

Annotations

Check warning on line 503 in src/tirith/platform/report.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add replacement fields or use a normal string instead of an f-string.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_aaw5M5V5vHQYD6&open=AaBtF_aaw5M5V5vHQYD6&pullRequest=291

Check failure on line 282 in src/tirith/platform/check.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 17 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_bUw5M5V5vHQYD8&open=AaBtF_bUw5M5V5vHQYD8&pullRequest=291

Check failure on line 716 in src/tirith/platform/check.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

LLMs running this code with faulty CLI arguments can escape file system restrictions. Refactor this code to validate the constructed path before accessing the file system.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_bUw5M5V5vHQYD_&open=AaBtF_bUw5M5V5vHQYD_&pullRequest=291

Check failure on line 94 in src/tirith/platform/report.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 20 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_aaw5M5V5vHQYD3&open=AaBtF_aaw5M5V5vHQYD3&pullRequest=291

Check warning on line 130 in src/tirith/platform/report.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused local variable "listed_resources".

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_aaw5M5V5vHQYD2&open=AaBtF_aaw5M5V5vHQYD2&pullRequest=291

Check failure on line 91 in src/tirith/platform/check.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

LLMs running this code with faulty CLI arguments can escape file system restrictions. Refactor this code to validate the constructed path before accessing the file system.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_bUw5M5V5vHQYD-&open=AaBtF_bUw5M5V5vHQYD-&pullRequest=291

Check failure on line 196 in src/tirith/tui/results.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 17 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_Xrw5M5V5vHQYD1&open=AaBtF_Xrw5M5V5vHQYD1&pullRequest=291

Check failure on line 232 in src/tirith/platform/report.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 28 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_aaw5M5V5vHQYD4&open=AaBtF_aaw5M5V5vHQYD4&pullRequest=291

Check failure on line 225 in src/tirith/platform/check.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

LLMs running this code with faulty CLI arguments can escape file system restrictions. Refactor this code to validate the constructed path before accessing the file system.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_bUw5M5V5vHQYEA&open=AaBtF_bUw5M5V5vHQYEA&pullRequest=291

Check failure on line 506 in src/tirith/platform/check.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 19 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_bUw5M5V5vHQYD9&open=AaBtF_bUw5M5V5vHQYD9&pullRequest=291

Check failure on line 414 in src/tirith/platform/report.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 20 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_aaw5M5V5vHQYD5&open=AaBtF_aaw5M5V5vHQYD5&pullRequest=291

Check failure on line 99 in src/tirith/platform/check.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 17 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_bUw5M5V5vHQYD7&open=AaBtF_bUw5M5V5vHQYD7&pullRequest=291

Check warning on line 134 in tests/platform/test_report_plan_attributes.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_bmw5M5V5vHQYEC&open=AaBtF_bmw5M5V5vHQYEC&pullRequest=291

Check failure on line 134 in src/tirith/plan_actions.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 26 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=StackGuardian_policy-framework&issues=AaBtF_bdw5M5V5vHQYEB&open=AaBtF_bdw5M5V5vHQYEB&pullRequest=291