Skip to content

Conversation

@zjwu0522
Copy link
Collaborator

@zjwu0522 zjwu0522 commented Sep 10, 2025

Change Type

  • ✨ feat
  • πŸ› fix
  • ♻️ refactor
  • πŸ’„ style
  • πŸ‘· build
  • ⚑️ perf
  • πŸ“ docs
  • πŸ”¨ chore

Description of Change

Fix Notion duplicate navigation by searching newest "<title> (1)" when URL is invalid
When Notion duplicates a page, the UI can occasionally navigate to the parent page instead of the new duplicate. This change adds a recovery path that:
Waits briefly (5s), then searches for the exact title "<title> (1)" in the source workspace.
Retries the search (3 attempts, 5s gaps) to account for eventual consistency.
Chooses the newest match and navigates to it, then continues the standard move/rename logic.
This makes duplication robust against occasional misnavigation without altering the existing flow.

Additional Information

@github-actions
Copy link

🐳 Docker Build Completed!

Version: pr-fix-notion-state-manager-59a0db1
Build Time: 2025-09-10T15:21:08.179Z
πŸ”— View all tags on Docker Hub: https://hub.docker.com/r/evalsysorg/mcpmark/tags

Pull Image

Download the Docker image to your local machine:

docker pull evalsysorg/mcpmark:pr-fix-notion-state-manager-59a0db1

Run Eval

Execute evaluation tasks using the built image:

DOCKER_IMAGE_VERSION=pr-fix-notion-state-manager-59a0db1 ./run-task.sh --models gpt-4.1-mini --tasks file_context/uppercase

Important

This build is for testing and validation purposes.

@xyliugo xyliugo merged commit aa73e5b into main Sep 10, 2025
3 checks passed
@xyliugo xyliugo deleted the fix/notion-state-manager branch September 10, 2025 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants