Skip to content

Remove loc search from agent guidance - #171

Merged
Saga4 merged 1 commit into
mainfrom
codex/remove-loc-search-guidance
Aug 5, 2026
Merged

Remove loc search from agent guidance#171
Saga4 merged 1 commit into
mainfrom
codex/remove-loc-search-guidance

Conversation

@Saga4

@Saga4 Saga4 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

For Quick fix, removing it from agent guidance as first thing, And in subsequent PR will make the cleanup.

Summary

  • remove loc search from generated agent skills, mount-local AGENTS.md, source guidance, and the onboarding prompt
  • steer agents toward mounted-folder browsing, normal filesystem tools, loc info, and loc locate for known Notion URLs/titles
  • update quickstart/best-practices examples while keeping the CLI reference and search implementation intact

Tests

  • cargo test -p loc-cli mount_writes_agent_guidance_and_claude_alias
  • cargo test -p localityd source_guidance_teaches_common_cli_workflow
  • cargo test -p locality-desktop --bin locality-desktop skill_teaches_connector_aware_locality_cli_workflow
  • cargo test -p locality-desktop --bin locality-desktop suggested_prompt_is_source_neutral_and_cli_first
  • cargo fmt --all -- --check
  • git diff --check

Note

  • I also tried cargo test -p locality-desktop agent_guidance; the changed tests passed, but the broader filter failed on agent_guidance::tests::no_supported_agent_fallback_names_locality_source_folder because this local machine detects supported agent tooling, so the fallback count is environment-dependent.

@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
locality Ready Ready Preview Aug 5, 2026 8:33pm

@Saga4
Saga4 merged commit d5c96bb into main Aug 5, 2026
10 of 12 checks passed
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.

1 participant