Skip to content

Emit references instead of definitions for properties on object literals that implement an interface #252

@olafurpg

Description

@olafurpg

Currently, scip-typescript emits a definition for property: 41 in cases like below

  interface Configuration {
    property: number
//  ^^^^^^^^ definition syntax 1.0.0 src/`infer-relationship.ts`/Configuration#property.
//  documentation ```ts\n(property) property: number\n```
  }
        return {
          property: 41,
//        ^^^^^^^^ definition syntax 1.0.0 src/`infer-relationship.ts`/property3:
//        documentation ```ts\n(property) property: number\n```
//        relationship implementation reference scip-typescript npm syntax 1.0.0 src/`infer-relationship.ts`/Configuration#property.
        }

Minimized reproduction in the tests
https://github.com/sourcegraph/scip-typescript/blob/main/snapshots/output/syntax/src/infer-relationship.ts#L29

We should change the behavior to emit a reference occurrence to src/infer-relationship.ts/Configuration#property. instead.

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions