Skip to content

Commit e7496bb

Browse files
committed
Use merged notification providers through normal Git dependencies
1 parent c059122 commit e7496bb

6 files changed

Lines changed: 94 additions & 98 deletions

File tree

‎.agents/docs/notifications.md‎

Lines changed: 11 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -6,26 +6,22 @@ Declare `notifications` in injection requirements. Field validation and persiste
66
business state stay beside the relevant control or object. Do not introduce a
77
custom Toast, page-wide operation error, or raw exception message.
88

9-
The [Host guide](https://github.com/cordisx/cordisx/blob/3158c0401a3f604727f623f1a8ac584017dd8540/.agents/docs/notifications.md)
9+
The [Host guide](https://github.com/cordisx/cordisx/blob/3cfe370eb7abf33e16686fbd82659cd441247fbd/.agents/docs/notifications.md)
1010
owns usage and interaction guidance; the
11-
[Protocol contract](https://github.com/cordisx/cordisx-protocol/blob/dfa2c5fa956184df11a97955b8e5c74a76cb8876/.agents/docs/notifications-v1.md)
12-
owns the public API. This migration is an experimental candidate, not a release.
11+
[Protocol contract](https://github.com/cordisx/cordisx-protocol/blob/f46dd21e15a949a26f05f89bf11dea339fc60c02/.agents/docs/notifications-v1.md)
12+
owns the public API. Source merge and package publication are separate stages.
1313

14-
## Reproduce the candidate SDK
14+
## Development dependencies
1515

16-
The local SDK archive is deliberately excluded from Git. Its exact Host commit,
17-
Protocol source and hashes are recorded in [notification-sdk-evidence.json](../../notification-sdk-evidence.json).
18-
From a Host checkout at `efbff656d84b482d51598bc5ba303d24134e0c62`, run the
19-
[SDK builder](https://github.com/cordisx/cordisx/blob/main/.agents/docs/sdk-source-packaging.md)
20-
into a fresh directory, then run from this package:
16+
`package.json` and `package-lock.json` pin formally merged Host and Protocol
17+
commits. Install through the repository's normal dependency flow:
2118

2219
```sh
23-
mkdir -p .cache/sdk
24-
cp /path/to/sdk/packages/cordisx-0.1.0-beta.2.tgz .cache/sdk/cordisx-efbff656d84b.tgz
25-
npm ci --ignore-scripts
20+
npm ci
2621
npm run check
2722
```
2823

29-
Compare the archive SHA-256 with the evidence before installation. Use one shared
30-
Protocol override; independent copies break branded public types. This explicit
31-
SDK build avoids recursive Git prepare scripts and does not change the API contract.
24+
Keep the shared Protocol override so public branded types resolve to one copy.
25+
A missing notification service requires a supported Host upgrade; do not recreate
26+
notification UI as an older-Host fallback. The former local candidate SDK archive
27+
is no longer an installation prerequisite.

‎AGENTS.md‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,7 @@ manually positioned alert, or page-wide success/error paragraph. Keep field
2121
validation and durable business state beside the relevant object. Use stable
2222
semantic `kind` values, localized safe text, and notification rules owned by Host;
2323
never expose raw exceptions or notify on every polling attempt.
24-
See the [Host notification guide](https://github.com/cordisx/cordisx/blob/3158c0401a3f604727f623f1a8ac584017dd8540/.agents/docs/notifications.md)
24+
See the [Host notification guide](https://github.com/cordisx/cordisx/blob/3cfe370eb7abf33e16686fbd82659cd441247fbd/.agents/docs/notifications.md)
2525
for the interaction contract and older-Host capability boundary.
2626

27-
The notification migration currently uses exact feature-branch SDK/Protocol
28-
revisions; it is not evidence of a formal release or a Mono pointer upgrade.
29-
30-
Candidate setup: [notification migration](./.agents/docs/notifications.md).
27+
Dependency setup: [notification migration](./.agents/docs/notifications.md).

‎README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,4 +42,4 @@ Host-owned implementation. This repository is licensed under
4242

4343
## Notification feedback
4444

45-
See [operation notifications and candidate SDK setup](./.agents/docs/notifications.md).
45+
See [operation notifications and development dependencies](./.agents/docs/notifications.md).

‎notification-sdk-evidence.json‎

Lines changed: 0 additions & 50 deletions
This file was deleted.

‎package-lock.json‎

Lines changed: 78 additions & 25 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎package.json‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,13 +38,13 @@
3838
},
3939
"devDependencies": {
4040
"@cordisx/eslint-config": "github:cordisx/cordisxmono#c63c2e8c2ba7e11502934a52ad2ce3734e804cdc",
41-
"@cordisx/protocol": "github:cordisx/cordisx-protocol#f3e18c925c34d90fdf203c158811cef10f7ebf57",
41+
"@cordisx/protocol": "github:cordisx/cordisx-protocol#f46dd21e15a949a26f05f89bf11dea339fc60c02",
4242
"@deepseek-ai/cordis": "4.0.1",
4343
"@deepseek-ai/schemastery": "3.18.1",
4444
"@projectwallace/stylelint-plugin": "0.7.0",
4545
"@typescript-eslint/parser": "8.69.0",
4646
"@vitejs/plugin-react": "6.1.0",
47-
"cordisx": "file:.cache/sdk/cordisx-efbff656d84b.tgz",
47+
"cordisx": "github:cordisx/cordisx#3cfe370eb7abf33e16686fbd82659cd441247fbd",
4848
"dprint": "0.57.1",
4949
"eslint": "9.39.4",
5050
"stylelint": "17.15.0",

0 commit comments

Comments
 (0)