perf(planning): restore the integrated benchmark gate #278

Open

nessar wants to merge 1 commits from feature/benchmark-performance into develop

pull from: feature/benchmark-performance

merge into: nessar:develop

nessar:develop

nessar:renovate/docker-29.x

nessar:feature/verify-trace-free-planning

nessar:feature/remove-decision-trace

nessar:renovate/angular-monorepo

nessar:renovate/typescript-7.x

nessar:feature/gitea-actions

nessar:feature/gitflow-ci

nessar:main

nessar:select-duration

nessar:dependabot/npm_and_yarn/typescript-6.0.3

nessar:dependabot/github_actions/docker/build-push-action-7

nessar:dependabot/github_actions/docker/login-action-4

nessar:dependabot/github_actions/docker/metadata-action-6

nessar:dependabot/github_actions/docker/setup-buildx-action-4

nessar:dependabot/github_actions/actions/upload-artifact-7

nessar:copilot/sub-pr-61-another-one

nessar:copilot/sub-pr-61-again

nessar:copilot/sub-pr-61

nessar:copilot/sub-pr-49

nessar:copilot/sub-pr-37

nessar:copilot/update-ci-cd-pipeline

nessar:copilot/refactor-duplicated-code

Conversation 0 Commits 1 Files Changed 9

+594 -118

nessar commented 2026-10-16 08:43:45 +02:00

Owner

Copy link

Copy source

The integrated benchmark previously exceeded its unchanged 300-second timeout. Reduce candidate-scoring and allocation work so the complete seven-scenario protocol finishes in 298.551 seconds locally, then restore benchmark success as a requirement for image publication and the aggregate CI check.

Evaluate objectives lazily in priority order, reuse calendar geometry and working-day prefixes, and share existing placements during assignments while detaching mutable metadata. Preserve search breadth, rule order, earned-entitlement chronology, outcomes, explanations and deterministic tie-breaks. Benchmark inputs, assertions, warm-ups, repetitions and resource limits remain unchanged.

Validation:

• Required benchmark: exit 0 with all seven warm-ups and 14 measured runs; 2048 MiB heap, no coverage, 300-second timeout.
• 633 functional tests with coverage; application/spec type checks, lint, formatting, CI policy/image tests and production build pass.
• Complete baseline comparisons match for all seven scenarios and 182 configuration cases; three permanent regression cases pin objective vectors and provenance.
• Native-worker browser completion, input editing, heartbeats and keyboard cancellation checks pass. Independent Standards and Spec reviews found no findings.

Evidence and reproduction: docs/benchmark/issue-78.md, including all 14 protocol rows. The local timeout margin is only 1.449 seconds; live CI and security scans are unverified. No physical-phone run was available, and calendar rendering can pause the main thread for about two seconds.

Refs #78. The related verification issue #77 / PR #273 remains open; this PR does not close either issue or change the #268 activation gate.

The integrated benchmark previously exceeded its unchanged 300-second timeout. Reduce candidate-scoring and allocation work so the complete seven-scenario protocol finishes in 298.551 seconds locally, then restore benchmark success as a requirement for image publication and the aggregate CI check. Evaluate objectives lazily in priority order, reuse calendar geometry and working-day prefixes, and share existing placements during assignments while detaching mutable metadata. Preserve search breadth, rule order, earned-entitlement chronology, outcomes, explanations and deterministic tie-breaks. Benchmark inputs, assertions, warm-ups, repetitions and resource limits remain unchanged. Validation: - Required benchmark: exit 0 with all seven warm-ups and 14 measured runs; 2048 MiB heap, no coverage, 300-second timeout. - 633 functional tests with coverage; application/spec type checks, lint, formatting, CI policy/image tests and production build pass. - Complete baseline comparisons match for all seven scenarios and 182 configuration cases; three permanent regression cases pin objective vectors and provenance. - Native-worker browser completion, input editing, heartbeats and keyboard cancellation checks pass. Independent Standards and Spec reviews found no findings. Evidence and reproduction: [docs/benchmark/issue-78.md](https://gitea.nessar.fr/nessar/vac-optimizer/src/commit/859a5f11052a8e2d71321b1a685177b7f44588b3/docs/benchmark/issue-78.md), including all 14 protocol rows. The local timeout margin is only 1.449 seconds; live CI and security scans are unverified. No physical-phone run was available, and calendar rendering can pause the main thread for about two seconds. Refs #78. The related verification issue #77 / PR #273 remains open; this PR does not close either issue or change the #268 activation gate.

nessar added 1 commit 2026-10-16 14:01:48 +02:00

perf(planning): restore the integrated benchmark gate ... cbde23f572

Reduce objective evaluation and calendar allocation work while preserving planner outcomes and deterministic explanations. Restore required benchmark gating and record the unchanged protocol, differential checks, and browser evidence for #78.

nessar force-pushed feature/benchmark-performance from 859a5f1105 to cbde23f572 2026-10-16 14:01:48 +02:00 Compare

Some required checks failed

Hide all checks

CI / policy (push) Successful in 7s

Details

CI / validate (push) Successful in 1m23s

Details

CI / test (push) Successful in 3m17s

Details

CI / benchmark (push) Failing after 5m52s

Details

CI / build (push) Successful in 1m31s

Details

CI / security (push) Successful in 1m39s

Details

CI / images (push) Skipped

Details

CI / policy (pull_request) Successful in 7s

Details

CI / validate (pull_request) Successful in 1m11s

Details

CI / test (pull_request) Successful in 3m17s

Details

CI / benchmark (pull_request) Failing after 5m57s

Details

CI / build (pull_request) Successful in 1m30s

Details

CI / security (pull_request) Successful in 1m40s

Details

CI / images (pull_request) Skipped

Details

CI / required (push) Failing after 2s

Details

CI / required (pull_request) Failing after 1s

Required

Details

Some required checks were not successful.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin feature/benchmark-performance:feature/benchmark-performance

git checkout feature/benchmark-performance

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

dependencies

docker

documentation

duplicate

enhancement

github_actions

good first issue

help wanted

invalid

javascript

needs-info

needs-triage

question

ready-for-agent

wontfix

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

renovate (renovate) Bot nessar (nessar)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: nessar/vac-optimizer#278