elastic / elastic/elastic-package

resolveExternalFields shouldn't search for fields

Open
#843 0 comments 0 reactions 0 assignees View on GitHub
Team:Ecosystem
Dominant language
Go
Stars
72
Forks
141
Avg merge
19h 42m
Merged PRs (30d)
55

Description

Spotted in: https://github.com/elastic/elastic-package/pull/840#discussion_r890943940

It doesn't seem to be a big issue if we consider only "integration" type packages, but it appears to be a problem when we consider "input" type packages. The current logic "knows" where fields can be found (`/fields` or `/data_stream//fields`) and it "globs" them there.

It seems to be a better idea if field locations are provided externally.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating resolveExternalFields and its callers, then trace how field paths are currently derived for integration and input packages. Define where callers should provide field locations and confirm that the existing package-root and data_stream paths are no longer searched implicitly.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
cli, tooling
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.