Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 18, 2026

The function fibonacciCoherenceBoost was renamed to applyFibonacciWeightedBoost in commit 6baedf5, but test file imports and call sites were not updated, causing test failures.

Changes

  • Updated import to use applyFibonacciWeightedBoost
  • Updated test suite describe block name
  • Replaced 11 function call sites throughout test cases
// Before
import { fibonacciCoherenceBoost } from '../vortex-wavespec';

describe('fibonacciCoherenceBoost', () => {
  test('should boost coherence at iteration 0', () => {
    const boosted = fibonacciCoherenceBoost(baseCoherence, 0);
    // ...
  });
});

// After
import { applyFibonacciWeightedBoost } from '../vortex-wavespec';

describe('applyFibonacciWeightedBoost', () => {
  test('should boost coherence at iteration 0', () => {
    const boosted = applyFibonacciWeightedBoost(baseCoherence, 0);
    // ...
  });
});

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…ightedBoost in tests

Co-authored-by: toolate28 <105518313+toolate28@users.noreply.github.com>
Copilot AI changed the title [WIP] Update to address changes based on feedback from PR #24 Update test function calls to match renamed applyFibonacciWeightedBoost Jan 18, 2026
Copilot AI requested a review from toolate28 January 18, 2026 00:56
@toolate28 toolate28 marked this pull request as ready for review January 18, 2026 01:01
Copilot AI review requested due to automatic review settings January 18, 2026 01:01
@toolate28 toolate28 merged commit d474770 into copilot/sub-pr-8-again Jan 18, 2026
5 checks passed
@toolate28 toolate28 deleted the copilot/sub-pr-24 branch January 18, 2026 01:01
@github-actions
Copy link

🌀 Agent Review: Coherence >60%. Ethical quantum sims validated. Ready for merge.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the vortex wavespec test suite to use the new applyFibonacciWeightedBoost API instead of the old fibonacciCoherenceBoost, resolving the test failures caused by the earlier rename.

Changes:

  • Updated the named import in vortex-wavespec.test.ts to applyFibonacciWeightedBoost.
  • Renamed the test suite describe block to reflect the new function name.
  • Replaced all test call sites and the type-check guard to use applyFibonacciWeightedBoost.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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