Refactor router transitions to prefer Navigation API with History fallback by Copilot · Pull Request #18 · EasyWebApp/cell-router

Copilot · 2026-06-30T20:25:53Z

This updates routing internals to use the Web Navigation API for client-side transitions instead of writing directly to history.pushState. The refactor preserves backward compatibility by keeping History API behavior as a fallback in non-supporting environments.

Navigation write path
- History.handleLink() now calls navigation.navigate(..., { history: 'push', state }) when available.
- History.handleForm() applies the same Navigation API path for GET form submissions.
- Legacy fallback still uses history.pushState(...) with equivalent state payload.
State/title restoration
- Route restoration now reads entry state from navigation.currentEntry.getState() first.
- If unavailable, restoration falls back to history.state to keep existing behavior stable.
Docs and focused coverage
- Updated README feature wording from history.pushState() to navigation.navigate().
- Added focused jsdom coverage for:
  - Navigation API link transitions
  - Navigation API GET form transitions
  - History API fallback behavior
  - Empty Navigation state falling back to legacy history state

const navigation = getNavigation();

if (navigation?.navigate) {
    navigation.navigate(path, {
        state: { title },
        history: 'push'
    });
} else {
    legacyHistory.pushState({ title }, '', path);
}

TechQuery

@copilot 我 issue 里写的 polyfill 是白写的？就是让你彻底抛弃老 API！

导入补丁说明文档里的类型包，然后直接用最新 API，文档里注明：用户如果要支持老浏览器，自行安装补丁。

Copilot · 2026-06-30T20:49:58Z

@copilot 我 issue 里写的 polyfill 是白写的？就是让你彻底抛弃老 API！

导入补丁说明文档里的类型包，然后直接用最新 API，文档里注明：用户如果要支持��...

已按要求改成仅使用最新 Navigation API，并移除了 History API 回退；同时引入了 navigation-api-types 类型包，并在 ReadMe 里注明老浏览器需由使用方自行安装 polyfill（示例：navigation-api-polyfill）。对应提交：564a7e4。

…2, KoApache 3 & other latest Upstream packages/actions [optimize] simplify Copilot codes

Copilot

Pull request overview

This PR refactors cell-router’s client-side navigation to prefer the Web Navigation API for route transitions and title/state restoration, updates documentation to reflect the new navigation mechanism, and refreshes the project’s tooling/CI configuration.

Changes:

Refactored routing internals in source/History.ts toward Navigation API interception and navigation.navigate()-based transitions.
Added a new jsdom-focused unit test suite for History.
Updated docs and project tooling (deps/version bump, pnpm workspace config, CI action/runtime updates).

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
`source/History.ts`	Switches routing transitions/restoration toward Navigation API events and state handling.
`test/History.spec.ts`	Adds jsdom tests intended to cover Navigation API navigation behavior.
`ReadMe.md`	Updates docs to reference `navigation.navigate()` and mentions Navigation polyfill for legacy support.
`tsconfig.json`	Adds Navigation/Jest type packages to TypeScript compilation context.
`package.json`	Bumps version/deps and adjusts Jest transform configuration.
`pnpm-workspace.yaml`	Introduces pnpm workspace configuration and build allowlist.
`.github/workflows/main.yml`	Updates CI action versions and Node/pnpm versions used in publish workflow.

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

+    protected shouldIntercept(event: any) {
+        if (!event?.canIntercept || event.downloadRequest || event.formData) return false;
+


+    it('should use Navigation API to submit GET forms', async () => {
+        const { History, RouterMode } = await loadHistory();
+
+        const history = new History('https://example.com', RouterMode.history);
+
+        history.navigate('/search?keyword=router', { title: 'Cell Router' });
+
+        expect(navigate).toHaveBeenCalledWith('/search?keyword=router', {
+            state: { title: 'Cell Router' },
+            history: 'push'
+        });
+        expect(history.path).toBe('/search?keyword=router');
+    });


Initial plan

4332a39

Copilot AI assigned Copilot and TechQuery Jun 30, 2026

Copilot started work on behalf of TechQuery June 30, 2026 20:26 View session

Copilot AI linked an issue Jun 30, 2026 that may be closed by this pull request

[refactor] replace History API with Navigation API #10

Open

Copilot AI added 5 commits June 30, 2026 20:30

refactor: use Navigation API for route transitions

aab8dcb

test: align form navigation state handling

e5fb1d4

refactor: finalize navigation state handling

e777472

test: cover empty navigation state fallback

44ad99b

chore: tighten navigation listener and test spy

195be7c

Copilot AI changed the title ~~[WIP] Refactor to replace History API with Navigation API~~ Refactor router transitions to prefer Navigation API with History fallback Jun 30, 2026

Copilot finished work on behalf of TechQuery June 30, 2026 20:37

Copilot AI requested a review from TechQuery June 30, 2026 20:37

TechQuery requested changes Jun 30, 2026

View reviewed changes

Copilot started work on behalf of TechQuery June 30, 2026 20:44 View session

refactor: require Navigation API for route transitions

564a7e4

Copilot finished work on behalf of TechQuery June 30, 2026 20:50

Copilot AI requested a review from TechQuery June 30, 2026 20:50

TechQuery requested changes Jun 30, 2026

View reviewed changes

Comment thread source/History.ts Outdated

Copilot started work on behalf of TechQuery June 30, 2026 22:34 View session

Copilot AI added 2 commits June 30, 2026 22:37

chore: switch Navigation API typings to dom-navigation

d9f3752

refactor: harden navigation state title extraction

04c2aea

Copilot finished work on behalf of TechQuery June 30, 2026 22:40

Copilot AI requested a review from TechQuery June 30, 2026 22:40

TechQuery added the enhancement Some improvements label Jul 1, 2026

TechQuery added 2 commits July 2, 2026 05:11

[migrate] upgrade to PNPM 11, Lint-Staged 17, Puppeteer 25, FS-Match …

7e39219

…2, KoApache 3 & other latest Upstream packages/actions [optimize] simplify Copilot codes

[refactor] replace Manual Link & Form listeners with Navigation API

07dadc8

TechQuery marked this pull request as ready for review July 1, 2026 22:06

Copilot AI review requested due to automatic review settings July 1, 2026 22:06

Copilot started reviewing on behalf of TechQuery July 1, 2026 22:06 View session

Copilot AI reviewed Jul 1, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Refactor router transitions to prefer Navigation API with History fallback#18

Refactor router transitions to prefer Navigation API with History fallback#18
TechQuery with Copilot wants to merge 11 commits into
mainfrom
copilot/refactor-replace-history-api

Copilot AI commented Jun 30, 2026 •

edited by pull-request-badge Bot

Loading

Uh oh!

TechQuery left a comment

Uh oh!

Copilot AI commented Jun 30, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		protected shouldIntercept(event: any) {
		if (!event?.canIntercept \|\| event.downloadRequest \|\| event.formData) return false;

Uh oh!

Uh oh!

Conversation

Copilot AI commented Jun 30, 2026 • edited by pull-request-badge Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

TechQuery left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Jun 30, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Jun 30, 2026 •

edited by pull-request-badge Bot

Loading