Skip to content

feat: rename finding#55

Open
secDre4mer wants to merge 1 commit intomasterfrom
fix/rename-finding
Open

feat: rename finding#55
secDre4mer wants to merge 1 commit intomasterfrom
fix/rename-finding

Conversation

@secDre4mer
Copy link
Contributor

No description provided.

@secDre4mer secDre4mer self-assigned this Feb 6, 2026
@secDre4mer secDre4mer marked this pull request as ready for review February 11, 2026 13:53
@secDre4mer secDre4mer requested a review from gremat February 11, 2026 13:54
// Subject is the object analysed by THOR.
Subject ReportableObject `json:"subject" textlog:",expand"`
// Subject is the object assessed by THOR.
Subject AssessableObject `json:"subject" textlog:",expand"`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You replaced the clunky term ReportableObject with a yet more abstruse one. When we touch it now, it may be the best time to review the name in detail. Apart from the weird name, I'm mainly bothered with the point of view: in the context of an Assessment, the subject of interest is the object that triggered the assessment, but this is already appropriately called subject. However, it's not required that the type reflects this point of view. When we look at the object from outside this context, i.e., more independently, then it's just some object. We can of course not call it object (or ThorObject or similar) so we need to find something specific about it. And there is: THOR observed this thing somewhere, that's why it is reported, together with an assessment on it. So I'm thinking Observation (or something along that line, maybe ObservedObject) might be a better term. What do you think?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants