Skip to content

Harden Isogloss architecture, performance, and security - #7

Open
owengregson wants to merge 57 commits into
mainfrom
codex/pr6-performance-security-hardening
Open

Harden Isogloss architecture, performance, and security#7
owengregson wants to merge 57 commits into
mainfrom
codex/pr6-performance-security-hardening

Conversation

@owengregson

Copy link
Copy Markdown
Owner

Summary

  • hardens the exact PR 6 head across correctness, robustness, performance, filesystem safety, protocol binding, browser isolation, and CI supply-chain controls
  • replaces per-fragment runtime functions with scalar straight-line emission and reentrancy-safe pooled wrappers
  • makes identifier preprocessing AST-native and bounded, indexes pure-region and call-graph analyses, and adds enforced performance budgets
  • adds fail-closed resource limits and exactness checks across source transforms, BPRF generation, planning, and browser worker inputs
  • binds custody signatures to complete requests, lineage, covers, and exact target representations; uses hiding commitments for owner-secret material
  • adds contained two-phase filesystem protection, symlink/hardlink and physical-alias rejection, staged publication, rollback, and whole-tree replacement
  • keeps the nonlocal planner unpublished until compile, lowering, and evidence can be emitted atomically and bound to the exact contract digest
  • pins all workflow actions and runtimes, consolidates Pages deployment, refreshes vulnerable transitive dependencies, and removes the stale npm lockfile
  • adds a 166-item first-principles performance and security backlog in docs/performance-security-hardening-backlog.md

Review baseline

This branch starts at PR 6 head commit 02c481d. The changes were driven by independent correctness/robustness, performance, and security audits, followed by adversarial post-fix verification.

Validation

  • bun run typecheck
  • bun run test: 191 passed, 0 failed, 34,449 assertions
  • bun run build: library, browser worker, and Next.js static site
  • bun audit --production: no vulnerabilities
  • bun install --frozen-lockfile --dry-run
  • full benchmark budget: 15,220-byte protected output; 2.94 ms per 50,000 protected calls; checksum agreement
  • YAML parse and git diff --check
  • npm pack --dry-run: 40.2 KB package, 169.5 KB unpacked

Notable measured improvements

  • pathological pure-region planning at 320 regions fell from roughly 4.55 s and 265 MB to 372 ms and 6.8 MB
  • SCC inventory and call-risk aggregation now scale effectively linearly in measured 500/1,000/2,000-unit cases
  • large-expression protected execution improved from about 0.56/3.20/10.07 microseconds per call to 0.135/0.669/1.47 at 20/80/160 operations
  • published package output is substantially smaller after keeping experimental planner internals off the root export surface

owengregson and others added 30 commits July 24, 2026 11:43
Design doc + runnable launch packet for an Opus 5 / Fable 5 parallel-workflow
ideation campaign that forces novelty by construction: forbidden-solution
lists, assumption inversion, cross-domain analogy seeding, and forced fusion.
Scope is the whole next-gen tool (seven tracks); blue-sky generation with a
separate feasibility/novelty tagging pass. Agent-facing prompts use neutral
product/IP-protection framing. Plan only; the workflow is opt-in.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 25, 2026 15:23

Copilot AI 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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

@github-actions

Copy link
Copy Markdown

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 1 package(s) with unknown licenses.
See the Details below.

License Issues

apps/web/package.json

PackageVersionLicenseIssue Type
next^15.5.21NullUnknown License

OpenSSF Scorecard

PackageVersionScoreDetails
npm/next ^15.5.21 UnknownUnknown

Scanned Files

  • .github/workflows/deploy.yml
  • apps/web/package.json
  • packages/ruam/package-lock.json

Copilot AI review requested due to automatic review settings July 25, 2026 15:25

Copilot AI 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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

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