feat(deno): migrate root package.json → deno.json imports (Class B, standards#253 longtail) - #15
Merged
Conversation
…tandards#253 longtail) excel-economic-numbers-tool is a ReScript-driven Office add-in (Excel + LibreOffice). The package.json had `@microsoft/office-js` as the lone runtime dep + `@rescript/core` + `rescript` as devDeps. Class B migration: - Add `npm:rescript@^12.0.0` + `npm:@rescript/core@^1.6.0` + `npm:@microsoft/office-js@^1.1.0` to `deno.json` `imports`. - Replace the broken bare `rescript build` in the `build` task with the canonical `deno run -A --node-modules-dir=auto npm:rescript@^12.0.0` pattern (the bare form would fail in Deno-only environments — Deno doesn't have npx). - Add `nodeModulesDir: "auto"` for rescript's install lifecycle. - Delete `package.json` (now redundant — `deno.json` covers all runtime + dev deps + scripts). Office host integration unaffected: `@microsoft/office-js` is loaded by the Office host from the served manifest, not from the local `node_modules/`. Deno's `--node-modules-dir=auto` regenerates the local `node_modules/` at build time for rescript's lifecycle hooks. Per per-repo follow-up tracker for standards#253 (campaign closed 2026-05-31, longtail tracked in project_estate_npm_to_deno_2026_05_28.md). Refs hyperpolymath/standards#253. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
🔍 Hypatia Security ScanFindings: 154 issues detected
View findings[
{
"reason": "Action perpolymath/standards/.github/workflows/governance-reusable.yml@main\n needs attention",
"type": "unpinned_action",
"file": "governance.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in boj-build.yml",
"type": "missing_timeout_minutes",
"file": "boj-build.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in casket-pages.yml",
"type": "missing_timeout_minutes",
"file": "casket-pages.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in casket-pages.yml",
"type": "missing_timeout_minutes",
"file": "casket-pages.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in ci.yml",
"type": "missing_timeout_minutes",
"file": "ci.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in ci.yml",
"type": "missing_timeout_minutes",
"file": "ci.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in ci.yml",
"type": "missing_timeout_minutes",
"file": "ci.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in ci.yml",
"type": "missing_timeout_minutes",
"file": "ci.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in ci.yml",
"type": "missing_timeout_minutes",
"file": "ci.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in ci.yml",
"type": "missing_timeout_minutes",
"file": "ci.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
}
]Powered by Hypatia Neurosymbolic CI/CD Intelligence |
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
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
excel-economic-numbers-toolis a ReScript-driven Office add-in (Excel + LibreOffice). Thepackage.jsonhad@microsoft/office-jsas the lone runtime dep +@rescript/core+rescriptas devDeps.Class B migration:
npm:rescript@^12.0.0+npm:@rescript/core@^1.6.0+npm:@microsoft/office-js@^1.1.0todeno.jsonimports.rescript buildin thebuildtask with the canonicaldeno run -A --node-modules-dir=auto npm:rescript@^12.0.0pattern (the bare form would fail in Deno-only environments — Deno doesn't have npx).nodeModulesDir: "auto"for rescript's install lifecycle.package.json(now redundant —deno.jsoncovers all runtime + dev deps + scripts).Office host integration unaffected:
@microsoft/office-jsis loaded by the Office host from the served manifest, not from the localnode_modules/. Deno's--node-modules-dir=autoregenerates the localnode_modules/at build time for rescript's lifecycle hooks.Test plan
deno task rescript:buildproduces equivalent .res.js output to priornpx rescript builddeno task buildchains rescript build + bundle as beforePer per-repo follow-up tracker for standards#253 (campaign closed 2026-05-31, longtail tracked in
project_estate_npm_to_deno_2026_05_28.md).Refs hyperpolymath/standards#253.
🤖 Generated with Claude Code