Skip to content

docs: track map helper extraction - #62

Open
vycdev2 wants to merge 1 commit into
vycdev:mainfrom
vycdev2:docs/track-map-helper-extraction
Open

docs: track map helper extraction#62
vycdev2 wants to merge 1 commit into
vycdev:mainfrom
vycdev2:docs/track-map-helper-extraction

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

Verification

  • cargo fmt --check — passed
  • cargo build — passed
  • cargo test -q — passed (full workspace)
  • git diff --check upstream/main...HEAD — passed

Risk

  • Low: documentation-only tracking and architecture clarification; no compiler or language behavior changes.

Refs #61

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

vycdev commented Aug 3, 2026

Copy link
Copy Markdown
Owner

The documentation change looks good, but this branch now conflicts with main after #30 updated the same stdlib progress area. Please rebase and preserve both the completed math-helper extraction text and this map-helper tracking link.

@vycdev2
vycdev2 force-pushed the docs/track-map-helper-extraction branch from 736dcca to ccc327a Compare August 5, 2026 06:04
@vycdev2

vycdev2 commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator Author

Rebased onto current main and preserved the implemented source-defined math helper status alongside the map-helper tracking links. Verification: cargo fmt --check, cargo build, cargo test -q, and git diff --check upstream/main...HEAD passed.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

@vycdev vycdev left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The map boundary should not become an exception to Jett's established source-owned stdlib rule. Please revise the documentation so that:

  • every public map.* declaration belongs in compiler-shipped .jett source;
  • compositional helpers have real Jett bodies;
  • storage, key equality, lookup/update, and iteration may delegate to private trusted runtime kernels, but those kernels are implementation details rather than public compiler-owned functions;
  • the compiler does not retain hardcoded knowledge of public map names or signatures;
  • the current compiler-backed surface is described as transitional technical debt, not a permanent namespace exception.

Issue #61 may remain a bounded first extraction slice, but the PR and tracker should state the final requirement that the complete public map namespace becomes source-owned and should identify follow-up work for any public operations that cannot move in this slice.

The branch also currently conflicts with main in docs/progress.md after #64. Please resolve that conflict while preserving both the math.sum tracker (#63) and the map extraction tracker (#61).

@vycdev2
vycdev2 force-pushed the docs/track-map-helper-extraction branch from ccc327a to a9a9836 Compare August 8, 2026 03:12
@vycdev2

vycdev2 commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator Author

Rebased onto current main, resolved the docs/progress.md conflict while preserving both #61 and #63, and revised the docs and #61 tracker to make the complete public map namespace source-owned. The updated boundary keeps only private trusted storage, key-equality, lookup/update, and iteration kernels, and identifies all remaining public operations as required follow-up.

Verification: cargo fmt --check, cargo build, cargo test -q, and git diff --check upstream/main...HEAD passed.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

vycdev commented Aug 10, 2026

Copy link
Copy Markdown
Owner

Reviewed: the map-helper extraction tracker is sound, but this branch now conflicts with current main after #56 landed. Please rebase onto main and preserve the merged string-stdlib/namespace documentation. #58, #60, and #62 also conflict with one another (including the active JSON extraction plan), so land them serially and rerun the documentation diff/link checks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants