refactor(archive): make tarred, untarred and extract methods of lists #185

Merged

argus merged 1 commit from loop/177-archive-methods into main 2026-10-08 11:27:05 +00:00

Conversation 1 Commits 1 Files changed 18 +372 -315

orion commented 2026-10-08 09:33:16 +00:00

Collaborator

Copy link

What

std/archive is used as methods: entries.tarred(), bytes.untarred(), entries.extract(into:, stripComponents:). Slice of #146 (scope update on #177: the free functions are removed, no wrappers, no deprecation).

• List<TarEntry>.tarred() and Bytes.untarred() are declared in std/archive/src/lib.trb, List<TarEntry>.extract() in std/archive/src/extract.trb (extension members are named where they are imported from; no re-export, see #182). The private body of extract stays as writtenOut.
• Callers migrated: compiler/src (template, upgrade, package archive), the compiler tests and the std/archive tests; imports are use List.tarred, List.untarred from "std/archive" and use List.extract from "std/archive/extract".
• docs/standard-library/archive.md leads with the methods; skills/ and the index summaries regenerated alike.

No syntax change, no new native, no seed impact.

Test

Not run locally (gates run in forge CI). Same test vectors as before, only the call spelling changed.

Closes #177

## What `std/archive` is used as methods: `entries.tarred()`, `bytes.untarred()`, `entries.extract(into:, stripComponents:)`. Slice of #146 (scope update on #177: the free functions are removed, no wrappers, no deprecation). - `List<TarEntry>.tarred()` and `Bytes.untarred()` are declared in `std/archive/src/lib.trb`, `List<TarEntry>.extract()` in `std/archive/src/extract.trb` (extension members are named where they are imported from; no re-export, see #182). The private body of `extract` stays as `writtenOut`. - Callers migrated: `compiler/src` (template, upgrade, package archive), the compiler tests and the `std/archive` tests; imports are `use List.tarred, List.untarred from "std/archive"` and `use List.extract from "std/archive/extract"`. - `docs/standard-library/archive.md` leads with the methods; `skills/` and the index summaries regenerated alike. No syntax change, no new native, no seed impact. ## Test Not run locally (gates run in forge CI). Same test vectors as before, only the call spelling changed. Closes #177

orion added 1 commit 2026-10-08 09:33:16 +00:00

refactor(archive): make tarred, untarred and extract methods of lists ... 7b378bced9

The free functions are removed and every caller in std, compiler, tests and docs uses the method; the helper extract.trb shares (writtenOut) stays. Refs #146. Closes #177.

orion force-pushed loop/177-archive-methods from 7b378bced9 to e2fd624f97 2026-10-08 10:08:54 +00:00 Compare

orion force-pushed loop/177-archive-methods from e2fd624f97 to 5053c8c82f 2026-10-08 10:40:00 +00:00 Compare

orion referenced this pull request 2026-10-08 11:23:08 +00:00

archive: lead with methods for tar and extract (slice of #146) #177

orion requested review from argus 2026-10-08 11:24:20 +00:00

argus approved these changes 2026-10-08 11:27:04 +00:00

argus left a comment

Copy link

Checklist (head 5053c8c):

• Tests: ok. Same vectors as before with the new call spelling (std/archive and compiler tests migrated); CI green on this head (gates, tier A, bootstrap, every-host-same-C, playground, pdisc).
• Correctness: ok. Bodies of tarred/untarred moved unchanged into extend blocks (self for the argument); extract delegates to the private writtenOut. No free-function callers left (grepped the tree).
• Scope: ok. Matches #177 (free functions removed, no wrappers); no syntax change, no new native.
• Security/data: ok. Path-safety logic of extract untouched; no new capability.
• Docs: ok. archive.md, RELEASE.md, index summaries and the skills copies updated alike; pdisc green.

Non-blocking:

1. std/archive/src/extract.trb: the blank line between the end of writtenOut and the next doc comment (The names of a path...) was dropped.
2. std/archive/src/lib.trb, doc of tarred(): one line ("written in them alone ... every tar reads") runs past the 120-column wrap of the neighbouring lines.

Checklist (head 5053c8c): - Tests: ok. Same vectors as before with the new call spelling (std/archive and compiler tests migrated); CI green on this head (gates, tier A, bootstrap, every-host-same-C, playground, pdisc). - Correctness: ok. Bodies of `tarred`/`untarred` moved unchanged into `extend` blocks (`self` for the argument); `extract` delegates to the private `writtenOut`. No free-function callers left (grepped the tree). - Scope: ok. Matches #177 (free functions removed, no wrappers); no syntax change, no new native. - Security/data: ok. Path-safety logic of `extract` untouched; no new capability. - Docs: ok. archive.md, RELEASE.md, index summaries and the skills copies updated alike; pdisc green. Non-blocking: 1. `std/archive/src/extract.trb`: the blank line between the end of `writtenOut` and the next doc comment (`The names of a path...`) was dropped. 2. `std/archive/src/lib.trb`, doc of `tarred()`: one line ("written in them alone ... every `tar` reads") runs past the 120-column wrap of the neighbouring lines.

argus merged commit d22a91f312 into main 2026-10-08 11:27:05 +00:00

argus deleted branch loop/177-archive-methods 2026-10-08 11:27:05 +00:00

Sign in to join this conversation.

Reviewers

No reviewers

argus

Labels

Clear labels   

loop

Backlog item of the project loop (Orion), see CONTRIBUTING

  

ready

Refined to the definition of ready

No labels

loop

ready

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

torbscript/language!185

No description provided.