Skip to content

Lark identity shape test fails on main: return_files.py builds an unfoldable identifier regex #7387

Closed

Description

@hhyykk

Preflight

  • I searched existing issues and discussions for this behavior.
  • This report is not an unpatched security vulnerability.
  • The report and attachments contain no credentials or private/internal material.

Issue origin

Observed or reproduced in a real environment.

LoopX version or commit

2de490e (current main)

Host or runtime surface

Not host-specific

LoopX area

Build, packaging, installer, or CI

Problem

On current main (2de490e), this test fails:

tests/architecture/test_lark_identity_shape_owner.py::test_no_lark_module_hides_an_unfoldable_identifier_construction

It reports one extra item:

loopx/extensions/lark/return_files.py:100 builds a Lark identifier regex from a value this scan cannot fold; declare it or route the site to the owner

This is not caused by PR #7386 (#7386). That PR changes only loopx/control_plane/collaboration/delegation_context.py and its test, and the same test fails without it.

Minimal reproduction

uv sync --extra test
uv run --extra test pytest tests/architecture/test_lark_identity_shape_owner.py::test_no_lark_module_hides_an_unfoldable_identifier_construction -q

Expected behavior

The architecture test passes on main: the Lark identifier construction at loopx/extensions/lark/return_files.py:100 is either declared as allowed or routed to its owner.

Actual behavior

The test fails with the single extra item shown above.

Sanitized diagnostics

tests/architecture/test_lark_identity_shape_owner.py::test_no_lark_module_hides_an_unfoldable_identifier_construction
loopx/extensions/lark/return_files.py:100 builds a Lark identifier regex from a value this scan cannot fold; declare it or route the site to the owner

Additional context

Please either declare this construction as allowed, as the test message suggests, or route it to its owner. We did not change any Lark code or this test.

Owners: @huangruiteng @steven-kid @songoow @loopx-agent

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions