fix(find): exclude file/package Symbol nodes; stop rendering path-FQN as 'java'

[HumanBean17]

Problem

While testing #405 (absence diagnosis) on tests/bank-chat-system/, jrag find --fqn-contains 'Assign' returned 18 garbage lines:

java  role=OTHER  symbol_kind=file
java  role=OTHER  symbol_kind=file
... (x18)
DevAssignmentController  @chat-core  module=chat-app  role=CONTROLLER  symbol_kind=class

(The plain jrag find 'Assign' → 0 symbol + next: jrag find --fqn-contains Assign hint is #405 working as designed — exact match by design at ladybug_queries.py:1001; the hint is the new absence nudge.)

Root cause (pre-existing, unrelated to #405)

Two defects combine in the --fqn-contains path:

1. find_v2 leaks file/package nodes. _symbol_where_from_filter has no structural-kind guard, so :Symbol nodes with kind='file' match. A file node's fqn is the filesystem path (chat-assign/.../DevAssignmentController.java), and s.fqn CONTAINS 'Assign' matches the PascalCase filename. search_lexical.py:209-210 excludes these on purpose — find_v2 was the gap.
2. The renderer prints the file extension. find_v2's SELECT omits s.name, so NodeRef.name is None; display_name falls through to simple_name(fqn) = fqn.rsplit('.', 1)[-1] = java (the extension).

Verified against the live index — file nodes carry a good name but simple_name(path-FQN) returns the extension:

name='ChatAssignApplication.java' | simple_name(fqn)='java' | kind=file | role=OTHER | module=''

Fix (two layers)

• Layer 1 (mcp_v2.py find_v2): exclude kind in ('file','package') from the symbol branch, mirroring search_lexical.py. Safe unconditionally — DeclarationSymbolKind (= class/interface/enum/record/annotation/method/constructor) excludes those values, so no filter can ever request them.
• Layer 2 (jrag_envelope.py simple_name): a path-shaped fqn (separator present, no space — route fqns carry a space) returns the basename instead of the extension.

Layer 1 stops the noise at the query; layer 2 is defense-in-depth so any file/package node reaching the renderer (query mode, traversal) shows its filename, not java.

Verification

Reproduced before / after on tests/bank-chat-system/:

• Before: 18 java role=OTHER symbol_kind=file lines polluting the top.
• After: clean declaration results only (DevAssignmentController, AssignConfiguration, AssignProperties, …).

DevAssignmentController  @chat-core  module=chat-app  role=CONTROLLER  symbol_kind=class
DevAssignmentController#simulateAssign  ...  symbol_kind=method
AssignConfiguration  @chat-assign  ...  role=CONFIG  symbol_kind=class
...

Tests

• New test_find_symbol_excludes_file_and_package_nodes (reproduces the chat-assign case via the bank-chat-system fixture) + strengthened the existing empty-filter test to assert file/package exclusion.
• New test_simple_name_file_path_fqn_returns_basename_not_extension (incl. route-fqn non-regression).

Subset run (mcp + package + analysis + absence + graph + index): all green except 5 pre-existing/environmental failures unrelated to this change:

• 4× test_installer.py::TestPR4IndexProgress — fail on master too (FileExistsError, test-isolation issue with leftover index dirs).
• 1× test_search_typo_has_absence_diagnosis — order-dependent in the big suite (passes in isolation and on master-in-file); uses search_v2 + vocab index, which this change does not touch.

🤖 Generated with Claude Code