Feat/access errors by SharonStrats · Pull Request #893 · SolidOS/solid-ui

SharonStrats · 2026-10-14T02:14:32Z

This PR does a major refactor of the code so that all the code is not in one file and many things were simplified.

It also adds error handling using the debug for console logs, displaying errors to the user, handling input errors up front. I'll explain a few details below as well I have included an example of how it will look to the user.

Behavior notes:
Adding new assures valid authorization targets before Save

When adding a new authorization the target must resolve to a supported access-control subject: agent/WebID, agent group, agent class, or origin, where supported. If the target cannot be verified, it is rejected and is not added to the pending list. If that happens in the UI it will say Could not verify this access target
If target resolution throws for adding input—for example, because classification or related lookup fails—the exception is caught. The error message will show and if no message this Could not verify this access target
When an error is received here the user can retype and the error is cleared.

All save errors appear in the dialog below

Prompt: Validate the input for each subjectType for agent and agentGroup make sure you can load the doc and see that it has a valid agent, agentGroup, agentClass type, validate origin based on format we have the check in solidlogic already add hostname. Co-authored-by: GPT-5.6 Luna <gpt-5.6-luna@openai.com>

Prompt: Create shared ACL view models instead of reconstructing them in multiple methods. then rerun unit and e2e tests. Co-authored-by: GPT-5.6 Luna <gpt-5.6-luna@openai.com>

…errors Prompt: When there are errors the results that were successful need to be shown to the user in the open dialog, but they still should not be able to retry unsuccessful updates. Co-authored-by: GPT-5.6 Terra <gpt-5.6-terra@openai.com>

Copilot

🟡 Changes recommended

Failed saves can leave misleading role state, stale validation errors, and duplicate close events.

4 open findings

Prevent duplicate close event dispatch · New
Reconcile roles after all change failures · New
Guard stale async validation errors · New
Fix contradictory revoke failure message · New

What changed in this PR

Adds granular access-control error handling while decomposing the modal into focused components.

Changes:

Reports individual save and validation failures.
Extracts access-target, grant-draft, and save logic.
Adds tests for partial failures and target resolution.

File	Description
`test/components/access-control-modal/buildAccessChanges.test.ts`	Tests change construction.
`test/components/access-control-modal/accessTargetResolver.test.ts`	Tests target resolution.
`test/components/access-control-modal/accessGrantDrafts.test.ts`	Tests grant drafts.
`test/components/access-control-modal.test.ts`	Expands modal integration tests.
`src/components/access-control-modal/types.ts`	Defines events, drafts, and errors.
`src/components/access-control-modal/helpers.ts`	Adds origin normalization.
`src/components/access-control-modal/buildAccessChanges.ts`	Builds queued ACL changes.
`src/components/access-control-modal/applyAccessChanges.ts`	Applies changes with per-item errors.
`src/components/access-control-modal/accessTargetResolver.ts`	Resolves and verifies targets.
`src/components/access-control-modal/accessGrantDrafts.ts`	Manages grant draft models.
`src/components/access-control-modal/AccessControlSharedAccess.ts`	Extracts shared-access UI.
`src/components/access-control-modal/AccessControlSharedAccess.styles.css`	Styles shared-access UI.
`src/components/access-control-modal/AccessControlModal.ts`	Coordinates saving and errors.
`src/components/access-control-modal/AccessControlModal.styles.css`	Retains modal-level styles.
`src/components/access-control-modal/AccessControlGeneralAccess.ts`	Extracts general-access UI.
`src/components/access-control-modal/AccessControlGeneralAccess.styles.css`	Styles general-access UI.
`src/components/access-control-modal/AccessControlErrors.ts`	Renders save failures.
`src/components/access-control-modal/AccessControlErrors.styles.css`	Styles failure details.
`src/components/access-control-modal/AccessControlAddAccess.ts`	Extracts grant-entry UI.
`src/components/access-control-modal/AccessControlAddAccess.styles.css`	Styles grant-entry UI.

🧠 Review effort: Balanced

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

…istry@5.0.2-0) (latest: rdflib@2.4.1)

Prompt: Pasted the errors from the ci into the chat - can you see why these errors would happen in ci but not locally and fix. Co-authored-by: GPT-5.6 Luna <gpt-5.6-luna@openai.com>

Copilot

🟡 Changes recommended

Validation races, discarded lookup messages, and non-unique draft identifiers can produce incorrect or missing UI state.

5 open findings

4 resolved since last review

🧠 Review effort: Balanced

+export function getAccessGrantId (authorization: AccessAuthorization): string {
+  return getAuthorizationSubjectEntries(authorization)
+    .map(subject => `${subject.type}:${subject.iri}`)
+    .sort()
+    .join('|')


-      this.failed = true
-    }
-    this.logPendingGrantFailures(failedGrants, true)
+    this.clearInputError()


+          action: 'input',
+          label: 'Access target',
+          targetUri: principal,
+          message: error instanceof Error ? error.message : 'Could not verify this access target'


+  } catch {
+    return undefined


+    const first: AccessAuthorization = {
+      agent: ['https://alice.example/profile#me'],
+      agentGroup: ['https://example/group'],
+      agentClass: [],
+      origin: ['https://example.test'],
+      mode: ['Read'] as AccessMode[]
+    }
+    const second = {
+      ...first,
+      agent: [...first.agent].reverse(),
+      agentGroup: [...first.agentGroup].reverse(),
+      origin: [...first.origin].reverse()
+    }


timea-solid

I looked at the code in general and it looks ok. A lot of great work!

SharonStrats and others added 7 commits October 13, 2026 18:04

add error handling

19ee222

Clean up

f90f2c0

Prompt: Create shared ACL view models instead of reconstructing them in multiple methods. then rerun unit and e2e tests. Co-authored-by: GPT-5.6 Luna <gpt-5.6-luna@openai.com>

code cleanup to prepare for refactor

ff093f7

basic refactor splitting the access modal

20aa9bf

more refactoring and cleanup

3ace3b4

SharonStrats requested a balanced review from Copilot October 14, 2026 02:14

SharonStrats self-assigned this Oct 14, 2026

SharonStrats added the enhancement label Oct 14, 2026

Copilot started reviewing on behalf of SharonStrats October 14, 2026 02:15 View session

Copilot AI reviewed Oct 14, 2026

View reviewed changes

SharonStrats added 3 commits October 14, 2026 13:51

adjust styles

59ce76f

address PR feedback

2b075d9

Merge branch 'staging' into feat/access-errors

ccf79f6

SharonStrats force-pushed the feat/access-errors branch from b549cae to ccf79f6 Compare October 14, 2026 03:07

SharonStrats marked this pull request as ready for review October 14, 2026 03:07

SharonStrats requested review from NoelDeMartin and timea-solid October 14, 2026 03:07

chore: update solidos dependencies (dev: solid-logic@6.0.1-1 pane-reg…

7a145ce

…istry@5.0.2-0) (latest: rdflib@2.4.1)

SharonStrats linked an issue Oct 14, 2026 that may be closed by this pull request

Sharing pane: Add error handling #894

Open

Fixed test issues in CI - they were not local though for some reason

90bafaf

Prompt: Pasted the errors from the ci into the chat - can you see why these errors would happen in ci but not locally and fix. Co-authored-by: GPT-5.6 Luna <gpt-5.6-luna@openai.com>

SharonStrats force-pushed the feat/access-errors branch from c41b3f1 to 90bafaf Compare October 14, 2026 03:31

SharonStrats requested a balanced review from Copilot October 14, 2026 03:36

Copilot started reviewing on behalf of SharonStrats October 14, 2026 03:36 View session

Copilot AI reviewed Oct 14, 2026

View reviewed changes

timea-solid approved these changes Oct 14, 2026

View reviewed changes

Navigation Menu

Repository navigation

Feat/access errors - #893

Feat/access errors#893
SharonStrats wants to merge 12 commits into
stagingfrom
feat/access-errors

SharonStrats commented Oct 14, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

timea-solid left a comment

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

SharonStrats commented Oct 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

timea-solid left a comment

Choose a reason for hiding this comment

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

SharonStrats commented Oct 14, 2026 •

edited

Loading