Commit e270ebb
fix(core): PHASE2_IMPLEMENTATION.md imports from the root barrel, not an undeclared subpath (#16205)
`packages/core/PHASE2_IMPLEMENTATION.md` sections 4 and 5 told readers to write
`import { … } from '@objectstack/core/security'`. `packages/core/package.json`
declares exactly two `exports` entries, `.` and `./logger`, with no `./security`
and no wildcard, so that specifier resolves for no consumer of the published
package. Measured live at this head:
ERR_PACKAGE_PATH_NOT_EXPORTED - Package subpath './security' is not defined
by "exports" in packages/core/package.json
with `@objectstack/core/logger` resolving on the same command as the control.
Both classes are real and both are exported from the ROOT barrel, reached by
`export * from './security/index.js'` in `packages/core/src/index.ts`. So the
symbols exist and only the path is wrong. Both lines now name `@objectstack/core`.
Verified against the built root entry that the repaired specifier is true:
`PluginPermissionManager` and `PluginSandboxRuntime` are both declared in
`dist/index.d.ts` and both load from `dist/index.cjs`, with the retired
`PluginSecurityScanner` absent as the negative control and `PluginHealthMonitor`
present as the positive one.
No contract change: `packages/core/package.json` is untouched. Adding a
`./security` entry would widen the published surface on no evidence of demand
and remains the maintainer's call.
Claude-Session: https://claude.ai/code/session_01ARYe3yQTQCUFm5qPYNgKaJ
Co-authored-by: Claude <noreply@anthropic.com>1 parent a3bbb8b commit e270ebb
1 file changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
158 | 158 | | |
159 | 159 | | |
160 | 160 | | |
161 | | - | |
| 161 | + | |
162 | 162 | | |
163 | 163 | | |
164 | 164 | | |
| |||
216 | 216 | | |
217 | 217 | | |
218 | 218 | | |
219 | | - | |
| 219 | + | |
220 | 220 | | |
221 | 221 | | |
222 | 222 | | |
| |||
0 commit comments