chore(debt): record auth archetype-layout + erp-sync CSV-parse debt (record-only)#112
Merged
Conversation
Records two deferred auth-layer gaps flagged in a user review (record-only, no refactor this pass): - (L) auth-better-auth / auth-workos declare Archetype-2 but place factories in top-level src/*.ts (not src/application/), types inline (not src/domain/), and export no src/testing/ fakes; justified src/ports/ + one-adapter omissions are undocumented. - (P) better-auth got the R0-R5 seamless passthrough program; WorkOS has no equivalent feature-surfacing/parity program — open decision (managed AuthKit has no plugin system, but org/SSO/SCIM/MFA surface is unsurfaced). Cross-refs seamless better-auth roadmap, AS2-CONSOLIDATION, AS7 metadata entries. Recorded at user direction during the JSR-readiness umbrella merge/cleanup pass. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018fq9V7ujx7e1rWXi57qkPG
tutorials/erp-sync/02-import-job parses CSV by hand (split-based) instead of @std/csv (simple case) or in-memory DuckDB read_csv via @duckdb/node-api (analytical case). Teaching surface propagates a pattern that mishandles quoted/escaped fields. Record-only at user direction; no tutorial edit. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018fq9V7ujx7e1rWXi57qkPG
rickylabs
marked this pull request as ready for review
June 22, 2026 21:29
This was referenced Jun 22, 2026
rickylabs
added a commit
that referenced
this pull request
Jun 25, 2026
…ld version zero-drift (#129) * chore(release-prep): research + plan for alpha.3 pre-release gate Two slices, one PR: (S1) add pages.yml to main (release-triggered docs redeploy, version read from manifest); (S2) de-hardcode CLI scaffold test version fixtures for bump-safety. Realizes task #112's release gate. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014gW4zfhMMQU6txC828ijct * chore(openhands): apply partial agent changes (agent outcome: failure) * chore(openhands): record run trace 28196492395-1 * chore(openhands): apply agent changes * chore(openhands): record run trace 28197363632-1 * test(cli): derive scaffold version from NETSCRIPT_RELEASE_VERSION + drift guard (S2) * ci(pages): deploy docs from main on release + push (S1) * chore(run): record release-prep S1+S2 commits + gate evidence --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com> Co-authored-by: OpenHands Bot <openhands@all-hands.dev>
This was referenced Jul 3, 2026
Closed
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.
Record-only: two deferred arch-debt entries (no code/tutorial changes)
Captured during the JSR-readiness umbrella merge/cleanup pass, at user direction ("record it in arch
debt for now and take care of that later"). This PR appends to
.llm/harness/debt/arch-debt.mdonly — no source, package, or tutorial edits. Fixes are scheduled under their own future runs.
1.
AUTH-ARCHETYPE-LAYOUT— auth backends' Archetype-2 layout + WorkOS parity (bdefe3b3)@netscript/auth-better-authand@netscript/auth-workosdeclare Archetype-2(Integration) but place composition-root factories in top-level
src/*.ts(notsrc/application/create-*.ts), keep domain types inline (notsrc/domain/), and export nosrc/testing/fakes. Thesrc/ports/omission is legitimate (the consumedAuthBackendPortlivesin
@netscript/plugin-auth-core) and the one-adapter shape correctly skipsadapters/, but neitheris documented in-package — which is what makes the layout read as confusing.
betterAuthOptionspassthrough (commit539b808b) plusthe tracked R0–R5 seamless better-auth integration roadmap. WorkOS got no equivalent
feature-surfacing program. Likely justified (WorkOS AuthKit is a managed service with no plugin
ecosystem), but its org/SSO/SCIM/MFA surface is unsurfaced — recorded as an open parity decision,
not a settled non-gap.
seamless better-auth integration roadmap,AS2-CONSOLIDATION, andAS7 documentation architecture metadataentries — those share the same packages, so one future refactorpass can close all of them together.
2.
TUTORIAL-CSV-PARSE— erp-sync import-job hand-rolls CSV (0c99caf5)tutorials/erp-sync/02-import-jobparses CSV with manual string splitting instead of@std/csv(simple case) or in-memory DuckDB
read_csvvia@duckdb/node-api(analytical case). As ateaching surface it propagates a pattern that mishandles quoted/escaped fields, embedded
commas/newlines, and BOM.
Notes
main(9c229624); arch-debt.md append-only (trivial merge).