feat(ids): add the auth-user atom builder and the shared I subject atom (P14) - #25
Open
jonathanprozzi wants to merge 1 commit into
Open
jonathanprozzi wants to merge 1 commit into
jonathanprozzi wants to merge 1 commit into
Conversation
…atom (P14) - Add auth-user-atom.ts: deterministic IntuitionAuthUser atom data from a hashed user id (authUserIdHash, createAuthUserAtomData, serializeAuthUserAtomData, authUserAtomDataHex, calculateAuthUserAtomId) with the derivation version pinned in the payload. - Add i-atom.ts: the shared first-person subject atom whose identity is the exact byte `I`; I_SUBJECT_ID is computed from calculateAtomId at module load and pinned by test. - Export the new surface from the package index and cover it in the public-entrypoints test. Ported from the private reference implementation at feat/intuition-app @ 1b877506d; source imports converted to the package's .js style. The OAuth atom context URL is deliberately left unchanged pending a ruling. Identity impact: additive only — calculateAtomId, calculateTripleId, createPredicateAtomData and every existing constant keep their bytes; the new constants reproduce the reference values exactly.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Child PR of the v1.1.0-alpha umbrella (#15). Tracking: ENG-15883.
Current (train)
@0xintuition/idshas no builder for the authenticated-user atom and no shared first-person subject atom.Need
The reference implementation (intuition-v2
feat/intuition-app@ 1b877506d) derives a deterministicIntuitionAuthUseratom from a hashed user id with the derivation version pinned in the payload, and uses a sharedIsubject atom whose identity is the exact byteI.Reconciliation
auth-user-atom.ts(authUserIdHash,createAuthUserAtomData,serializeAuthUserAtomData,authUserAtomDataHex,calculateAuthUserAtomId) andi-atom.ts(I_SUBJECT_IDcomputed fromcalculateAtomIdat module load and pinned by test), exported from the index and covered by the public-entrypoints test..jsstyle.calculateAtomId,calculateTripleId,createPredicateAtomDataand every existing constant keep their bytes; the new constants reproduce the reference values exactly.Held out
schema.intuition.systems; the reference usesschema.0xintuition.com. Changing it changes OAuth atom bytes and ids, so this PR keeps the train value. Reviewers: please confirm which host is live; that decides whether a follow-up changes it.Evidence
Review notes
CI shows the base branch's expired supply-chain exception until the CI-fix child PR merges; this branch is rebased after that.