feat(projection): add ExecsByPath composite-key surface to ProjectedContainerProfile

[entlein]

Adds ExecsByPath map[string][]string to ProjectedContainerProfile, populated by projection_apply.Apply() from cp.Spec.Execs. Nil source Args projects to an empty-but-non-nil slice, so the map distinguishes "path never recorded" (key absent) from "path recorded with no args" (key present, []string{}).

Required by node-agent#807. No new imports, no storage bump. Tests cover nil→empty projection, cloned-slice (no source aliasing), and last-write-wins on duplicate paths.

[coderabbitai]

Warning

Rate limit exceeded

@entlein has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 58 minutes and 40 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7268a41a-9476-4abc-b50d-61f0a50fc2e3

📥 Commits

Reviewing files that changed from the base of the PR and between cc59fa0 and 58ce8d9.

📒 Files selected for processing (3)

• pkg/objectcache/containerprofilecache/projection_apply.go
• pkg/objectcache/containerprofilecache/projection_apply_test.go
• pkg/objectcache/projection_types.go

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[entlein]

@coderabbitai review

Collective re-review request after applying your atomic-split advisory + rabbit-feedback fixes across the full SBOB upstream PR set. Each PR is now scoped to a single tier per your dependency map:

Tier 0 (opens):

• path-wildcards: anchored trailing-* + per-endpoint port + R0040 args storage#323 (path-wildcards core: anchored trailing-*, per-port key, WildcardIdentifier + CollapseConfig API — squash per your "merge first / or squash together" note)
• feat(cel): was_path_opened* CEL helpers in applicationprofile library #811 (was_path_opened* CEL helpers — NEW)

Tier 2 (execs):

• allowing wildcards in exec args storage#322 (CompareExecArgs)
• profile-compaction: CollapseConfig CRD + projection overlay + user-maged lifecycle storage#325 (CollapseConfig CRD + processors)
• allowing wildcards in exec args #805 (resolveExecPath + 3-arg parse)
• path-wildcards: anchored trailing-* + per-endpoint port + R0040 args #807 (R0040 args consumer)
• feat(projection): add ExecsByPath composite-key surface to ProjectedContainerProfile #812 (ExecsByPath composite-key surface — NEW)

Tier 3 (network):

• allowing CIDRs, wildcards and Plural in IP and DNS storage#324 (networkmatch package)
• allowing CIDRs, wildcards and Plural in IP and DNS #806 (network wildcards CEL)

Tier 4 (tamper/sign):

• profile-compaction: CollapseConfig CRD + projection overlay + user-ma… #808 (projection overlay + tamper + non-blocking fanout + NN resource)
• sign and detect tampering #809 (cosign sign/verify + tamper detection)

Umbrellas:

• SBOB wildcards: DNS + IP + path exec wildcards, sign/tamper  storage#326
• SBOB wildcards: DNS + IP + path exec wildcards, sign/tamper  #810

Please confirm atomicity per your earlier advisory and flag any remaining cross-PR coupling we missed.

[coderabbitai]

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Fetch PR `#812` details and diff
gh pr view 812 --repo kubescape/node-agent --json title,body,files,baseRefName,headRefName,state

# List changed files
gh pr diff 812 --repo kubescape/node-agent --name-only

# Get the actual diff for review
gh pr diff 812 --repo kubescape/node-agent

Length of output: 9271

---

@entlein — full diff reviewed. Atomicity of #812 is confirmed: purely additive, zero coupling beyond being the declared dependency of #807. A few things to address before merge:

---

pkg/objectcache/projection_types.go

Field placement — ExecsByPath is inserted after IngressAddresses and before SpecHash/SyncChecksum, separated from the logical exec group. Grouping it with other exec-surface fields improves readability and makes the struct self-documenting:

// near Execs / exec-related fields
Execs           ProjectedField
ExecsByPath     map[string][]string   // add here, not after ingress fields

---

pkg/objectcache/containerprofilecache/projection_apply.go

Last-write-wins data loss — this is the most material concern. When two ExecCalls entries share a Path but carry different Args slices, only the second survives in ExecsByPath, while Execs.Values (fed by extractExecsPaths) retains both entries. A consumer that gates on Execs ("is this path allowed?") and then looks up ExecsByPath ("what args were seen?") will silently miss the first entry's args. Consider one of:

1. Accumulate — change the value type to [][]string (path → list of observed arg slices) so no entry is dropped. Breaking change to the field type, but catches the semantic mismatch cleanly.
2. Union — flatten all args across duplicates into one []string, deduped. Lossy but simple.
3. Document explicitly and let callers decide. If the storage layer guarantees at most one ExecCalls per path per profile, add an invariant comment + a test that asserts extractExecsPaths and extractExecsByPath produce the same key-set cardinality.

Capacity hint — make(map[string][]string, len(cp.Spec.Execs)) is a valid upper-bound hint even when duplicates collapse. No change needed, but worth a comment so future readers don't "fix" it.

---

pkg/objectcache/containerprofilecache/projection_apply_test.go

sourceCopy ordering dependency — in TestApply_ExecsByPath_PopulatesFromSpec, sourceCopy is captured after Apply() returns:

pcp := Apply(&objectcache.RuleProjectionSpec{}, cp, nil)
// ...
sourceCopy := append([]string{}, cp.Spec.Execs[2].Args...) // ← post-Apply
pcp.ExecsByPath["/bin/sh"][0] = "MUTATED"
assert.Equal(t, sourceCopy, cp.Spec.Execs[2].Args, ...)

This is currently safe because Apply is documented as a pure transform and does not mutate cp. But the test's clone-invariant check is supposed to verify that purity; capturing sourceCopy after Apply makes the test pass vacuously if Apply had already mutated the source. Capture before Apply to make the invariant self-proving:

// Capture BEFORE Apply so the clone-invariant check is non-vacuous.
sourceArgs := append([]string{}, cp.Spec.Execs[2].Args...)

pcp := Apply(&objectcache.RuleProjectionSpec{}, cp, nil)
require.NotNil(t, pcp)

pcp.ExecsByPath["/bin/sh"][0] = "MUTATED"
assert.Equal(t, sourceArgs, cp.Spec.Execs[2].Args,
    "mutating the projected slice must not propagate to the source profile (cloned, not aliased)")

---

Tier-map atomicity confirmation

PR	Verdict	Notes
#812 (this)	✅ atomic	Purely additive; no new imports; no changes to existing behaviour
#807 → #812	✅ clean dep	#807 needs ExecsByPath present; #812 is the sole provider
#808 → #812	⚠️ needs rebase note	#808 must drop its own ExecsByPath addition to avoid a merge conflict after #812 lands; confirm #808's PR description calls this out
#812 → storage	✅ no coupling	No new storage symbols imported; compiles against storage main today

No other cross-PR coupling found in the files touched by #812.

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.