Skip to content

prepare npm 1.1.0 package contract#22

Merged
rebel0789 merged 1 commit into
mainfrom
codex/npm-1.1.0-package-contract
Jul 15, 2026
Merged

prepare npm 1.1.0 package contract#22
rebel0789 merged 1 commit into
mainfrom
codex/npm-1.1.0-package-contract

Conversation

@rebel0789

Copy link
Copy Markdown
Owner

What changed

  • make bundled benchmark datasets work from a globally installed package
  • install and exercise the real packed tarball in consumer smoke coverage
  • verify the README setup, memory, semantic, graph, MCP, handoff, benchmark, and local app flows
  • exclude checkout-only Node tests from the npm artifact and enforce size budgets
  • make release copy accurate before and after the registry handoff
  • regenerate repository and release evidence

Why

The previous consumer smoke mostly invoked source files directly. It could pass even when the npm artifact was missing a runtime file. Benchmark commands also looked for bundled fixtures in the user's repository instead of the installed package.

Impact

The 1.1.0 tarball now has 828 files, is about 1.21 MB compressed, and is about 5.21 MB unpacked. The previous artifact had 907 files, was about 1.46 MB compressed, and was about 6.57 MB unpacked. Runtime code, public docs, benchmark fixtures, and Rust source remain included.

Verification

  • npm run ci
  • npm run consumer:smoke
  • npm run consumer:browser-smoke
  • npm run native:smoke
  • npm run release:readiness:check
  • npm audit --omit=dev --audit-level=low
  • npm pack --json

@rebel0789
rebel0789 marked this pull request as ready for review July 15, 2026 14:09

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 83169ed221

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/cli/oaf.mjs
Comment on lines +4629 to +4632
const selectedPath = requestedPath ?? bundledPath;
const workspacePath = path.resolve(selectedPath);
const workspaceFile = await stat(workspacePath).catch(() => null);
if (workspaceFile?.isFile()) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Prefer package fixtures when no dataset is supplied

When an installed user runs a default benchmark such as recall bench temporal --read-only from a target repository that happens to contain evals/temporal/gold.v1.json (or the matching sufficiency/LoCoMo path), this workspace probe wins before the packaged fixture is considered, even though the user did not pass --dataset. That lets unrelated repo files change the supposedly fixture-backed benchmark result and dataset.ref, making the package contract non-deterministic; only explicit --dataset values should be resolved from the workspace before falling back to PACKAGE_ROOT.

Useful? React with 👍 / 👎.

@rebel0789
rebel0789 merged commit 06a947f into main Jul 15, 2026
3 checks passed
@rebel0789
rebel0789 deleted the codex/npm-1.1.0-package-contract branch July 15, 2026 14:14
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.

1 participant