Skip to content

[Feature request] Add aliases to workspace symbols #263

Description

@baraahomsi

Right now, only link completion suggests aliases. It would be nice if they were returned by workspace symbols too, e.g. from Telescope or vim.lsp.buf.workspace_symbol() in Neovim.

Activity

  1. Feel-ix-343 commented on Jun 30, 2025

    @Feel-ix-343
    Owner

    Oh yea this is a good idea!

    Would you be up for implementing this?

  2. algora-pbc commented on Aug 12, 2025

    @algora-pbc

    💎 Feel-ix-343 is offering a $5 bounty for this issue. View and reward the bounty at algora.io/Feel-ix-343/markdown-oxide/issues/263

    👉 Got a pull request resolving this? Claim the bounty by commenting /claim #263 in your PR and joining algora.io

  3. added a commit that references this issue on Mar 10, 2026
  4. github-actions commented on Mar 10, 2026

    @github-actions

    🤖 Devin session started for this issue!

    View Devin session

  5. zzhang82 commented on May 11, 2026

    @zzhang82

    Hi! Checking in to see if this is still available to work on. I noticed the previous intent comment from ~3 weeks ago and the open PR #435. If the current PR is stalled or you would like a fresh pair of eyes on this implementation, I would be happy to help out.

  6. slendereater-sketch commented on May 20, 2026

    @slendereater-sketch

    /attempt #263

  7. sk8kpwhrjt-creator commented on May 21, 2026

    @sk8kpwhrjt-creator

    /attempt #263

    Plan: add actual workspace-symbol alias entries, not only regression coverage. I will trace the existing metadata alias model and symbol provider, add focused tests showing aliases are returned alongside the canonical file symbol, and submit a narrow PR after cargo fmt/test.

  8. sk8kpwhrjt-creator commented on May 21, 2026

    @sk8kpwhrjt-creator

    Update after reading current main: alias symbol emission is already implemented in Vault::to_symbol_informations, and the open PRs #435/#446 are covering this with regression tests. I am not going to add a duplicate PR for the same test-only slice.

  9. luisalias007-cmyk commented on May 30, 2026

    @luisalias007-cmyk

    /attempt #263

    I am adding focused regression coverage that constructs a real vault with frontmatter aliases and verifies workspace symbols return those aliases both for empty client queries and alias searches.

  10. b3417 commented on Jun 12, 2026

    @b3417

    /attempt #263

  11. b3417 commented on Jul 4, 2026

    @b3417

    Pushed a formatting cleanup to the existing PR and verified the alias regression test still passes locally. The branch is back in a clean state on my side.

  12. stffinfcti commented on Sep 20, 2026

    @stffinfcti

    /attempt #263

    Plan: aliases are already emitted by Vault::to_symbol_informations on main; the missing piece is regression coverage proving they reach workspace/symbol clients. I will add a focused test that builds a real vault with frontmatter aliases and verifies aliases are returned for both the empty-query listing path (Telescope/vim.lsp.buf.workspace_symbol()) and alias-name queries, while filename queries still resolve to the canonical symbol.

  13. projectcarbonfiber commented on Sep 21, 2026

    @projectcarbonfiber

    /attempt #263

    Plan: alias emission already lives in Vault::to_symbol_informations on main. Open PRs #435/#463/#467/#470 only assert symbol names. This attempt adds picker-facing container_name on alias symbols, skips empty/duplicate aliases, strengthens regression coverage (empty query, fuzzy alias query, URI/kind parity, no-alias files), and updates the docs so workspace symbols mention frontmatter aliases.

  14. projectcarbonfiber commented on Sep 21, 2026

    @projectcarbonfiber

    /attempt #263

    Plan: aliases already emit from Vault::to_symbol_informations on main. Differentiating by setting container_name on alias symbols to the canonical refname (so Telescope / vim.lsp.buf.workspace_symbol() pickers show where the alias points), skipping blank/duplicate aliases, normalizing nested-note separators, and adding regression coverage for empty-query listing plus alias fuzzy search.

  15. themaksat commented on Sep 21, 2026

    @themaksat

    I have submitted a solution for this issue in PR #518. Full implementation and verification included. Payout wallet (Solana): 2BTCUUTviLNDtZDRBhPBTwSXZMrTttPJNbGevMXBBMKK

  16. wspperrimh commented on Oct 4, 2026

    @wspperrimh

    /claim #263

    Submitted in PR #533. Note: this repo requires maintainer approval before fork-PR workflows run, so CI shows "action required" until approved — the branch passes cargo test (80 tests) and cargo fmt --check locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions