chore(dlc): remove vendored DLC subtrees post-migration#70
Merged
Conversation
…reversibley} The idaptiky and idaptik-reversibley DLCs were vendored into this repo in commit 0c863ac (2026-04) because their dedicated GitHub repos held only scaffolds. The May-2026 migration PRs (idaptik-dlc-iky#1 merged 94a52e9; idaptik-dlc-reversibley#1 merged 37d5480c) imported the vendored content as the initial real content of each port. With both ports now real, the vendored copies here are pure duplication and are removed. Also drop the corresponding stale .gitmodules entries (which pointed at gitlab URLs that no longer matched the current GitHub-hosted ports). Side effect for the TypeScript-elimination sweep: drops 52 .ts files from this repo's footprint, taking idaptik from 52 to 0 .ts files. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
hyperpolymath
added a commit
that referenced
this pull request
May 2, 2026
Filed across the AffineScript ecosystem in response to UPSTREAM-CASCADE (commit 98f110c). Each PR cross-references the relevant idaptik artefacts as evidence. stapeln (3): #18 — fix(cerro-torre): pin Alire URL to explicit version (closes #13) #19 — fix(vordr): commit Cargo.lock for reproducible builds (closes #14) #20 — fix(rokur): use deno binary release directly (closes #16) affinescript (4): #62 — docs(AI.a2ml): 4 patterns + 3 directives (closes #45, #46, #47, #48, #49, #50, #52) #67 — feat(parser): accept [T] array/list shorthand (closes #40) — highest single-fix leverage in the entire cascade per UPSTREAM-CASCADE; ~95% of idaptik's translatable surface #69 — docs(human-guide): migration chapter + refinement-args + affine-async (closes #51, #53, #54, #55) #70 — docs(effects): migration-stance doc (closes #41) Total: 7 PRs, 16 issues to be closed when merged. Intentionally not addressed (sub-day budget): - stapeln#12 (selur Containerfile) — write new multi-stage - stapeln#15 (svalinn npm/Deno) — design decision needed - affinescript#35 (Node+vscode backend) — multi-week new target - affinescript#39 (Float operators) — typechecker design - affinescript#42 (extern fn/type) — multi-piece, untested locally - affinescript#56 (DOM/Pixi prod-ready) — library scale, gated on #42 - affinescript#57 (Migration assistant) — multi-week new tool - affinescript#58 (proven shape) — sibling repo work - affinescript#59 (Effect-row names) — design discussion Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.
Summary
Drop the two vendored DLC subtrees from this repo now that the ports hold the real content.
dlc/idaptiky/(172 files, including 52.ts)dlc/idaptik-reversibley/(76 files)Plus stale
.gitmodulesentries pointing at GitLab URLs (the original submodule destinations, superseded when the trees were vendored in 0c863ac).Why now
Both port repos held only scaffolds (CHANGELOG/COC/LICENSE/README/SECURITY) until earlier today, when these PRs imported the vendored content as the initial real content:
94a52e9)37d5480c)With both merged, the vendored copies here are pure duplication.
Side effect: TS-elimination sweep
This drops 52
.tsfiles from the idaptik repo's footprint (52 → 0). All 52 lived inside the two subtrees being removed; no straggler TypeScript files remain in idaptik after this lands. Estate-wide TS goes from 79 to 27 (the 27 are documented exemptions:.d.tspublic APIs, VS Code extensions blocked on #35, MCP/SDK adapters that need to stay TS).Caveat: workflows
Both port migration PRs stripped
.github/workflows/*because the OAuth token used to push lacked theworkflowscope. Those workflows still need to be added to each port repo (via web UI or after granting scope). Until then, the ports won't have CI.Test plan
dlc/idaptiky/ordlc/idaptik-reversibley/(already grep-checked clean except the .gitmodules entries removed in this PR)🤖 Generated with Claude Code