Skip to content

feat: permissions revamp #2398

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
golanglemonade merged 9 commits into from
May 26, 2026

matching

a18cb4e
Select commit
Loading
Failed to load commit list.
Merged

feat: permissions revamp #2398

matching
a18cb4e
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed May 26, 2026 in 1m 42s

Quality Gate failed

Failed conditions
10.9% Coverage on New Code (required ≥ 20%)

See analysis details on SonarQube Cloud

Annotations

Check failure on line 22 in internal/ent/hooks/orgownedtuples.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5B4NGfSosnWSdcrVb7&open=AZ5B4NGfSosnWSdcrVb7&pullRequest=2398

Check failure on line 41 in fga/generate/schemaparse/policy.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 30 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5kr--7KuT39oMgGdmT&open=AZ5kr--7KuT39oMgGdmT&pullRequest=2398

Check failure on line 217 in internal/ent/privacy/rule/orgowner.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "user does not have access to the requested organization" 3 times. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5B4NHeSosnWSdcrVb9&open=AZ5B4NHeSosnWSdcrVb9&pullRequest=2398

Check failure on line 99 in fga/generate/modelparse/parse.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 42 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5kr-5eKuT39oMgGdmS&open=AZ5kr-5eKuT39oMgGdmS&pullRequest=2398

Check warning on line 125 in fga/model/helpers.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Group together these consecutive parameters of the same type. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5B4NTfSosnWSdcrVcE&open=AZ5B4NTfSosnWSdcrVcE&pullRequest=2398

Check failure on line 25 in fga/generate/modelparse/parse.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5kr-5eKuT39oMgGdmR&open=AZ5kr-5eKuT39oMgGdmR&pullRequest=2398

Check failure on line 207 in internal/ent/privacy/rule/orgowner.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5B4NHeSosnWSdcrVb_&open=AZ5B4NHeSosnWSdcrVb_&pullRequest=2398

Check failure on line 133 in fga/generate/schemaparse/policy.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 18 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5kr--7KuT39oMgGdmU&open=AZ5kr--7KuT39oMgGdmU&pullRequest=2398

Check failure on line 25 in internal/ent/hooks/objectownedtuples.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 36 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5B4NBwSosnWSdcrVb6&open=AZ5B4NBwSosnWSdcrVb6&pullRequest=2398

Check failure on line 129 in fga/generate/schemaparse/parse.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5kr-_HKuT39oMgGdmV&open=AZ5kr-_HKuT39oMgGdmV&pullRequest=2398

Check warning on line 4 in fga/model/helpers.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a comment explaining why this blank import is needed. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5B4NTfSosnWSdcrVcD&open=AZ5B4NTfSosnWSdcrVcD&pullRequest=2398

Check failure on line 38 in fga/generate/main.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 37 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5B4NToSosnWSdcrVcH&open=AZ5B4NToSosnWSdcrVcH&pullRequest=2398

Check failure on line 27 in internal/ent/privacy/rule/orgowner.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 23 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5B4NHeSosnWSdcrVb-&open=AZ5B4NHeSosnWSdcrVb-&pullRequest=2398

Check failure on line 30 in internal/ent/privacy/rule/organization.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5B4NHzSosnWSdcrVcA&open=AZ5B4NHzSosnWSdcrVcA&pullRequest=2398

Check warning on line 466 in internal/graphapi/apitoken_test.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Group together these consecutive parameters of the same type. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5B4NMGSosnWSdcrVcB&open=AZ5B4NMGSosnWSdcrVcB&pullRequest=2398

Check failure on line 269 in internal/ent/privacy/rule/orgowner.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "requested object not in organization" 3 times. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ5B4NHeSosnWSdcrVb8&open=AZ5B4NHeSosnWSdcrVb8&pullRequest=2398
View more details on SonarQubeCloud