fix(agentic-ai): stale activity references and a dependency-level fix - #84
Open
dzeusking-dev wants to merge 3 commits into
Open
Conversation
…e to level 3 in the dimension table
…s-reference to canonical titles
…de to level 3 and fix stale dependsOn comment
Contributor
|
Hi @dzeusking-dev , thank you for the PR. I agree with everything except the level of static and dynamic tests. Kind regards |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
This is a focused review of the Agentic AI dimension, concentrating on the
areas highlighted for it: activity titles, maturity levels, dependency structure
and the human-verification ladder. The dimension introduced in #80 (and the
activities added afterwards) is in good shape; this PR does not rewrite it. It
corrects two concrete internal-consistency defects found during a focused
review of the Agentic AI dimension and its dependencies, while leaving the
substantive security guidance untouched. It does not claim to close #80.
Review method
source YAML (title, uuid, level, subdimension, tags, dependencies,
implementation refs, SAMM/ISO mappings), rather than trusting the
documentation.
AI activities and their dependencies: uuid and activity-name consistency,
dependency resolution and level progression, cycle detection, implementation
$refresolution, required fields / numeric ranges / tags, anddependsOninline comments vs. canonical activity titles.
schemas/dsomm-schema-agentic-ai.json.ARCHITECTURE.mdinventory against the canonical YAML.generated model.
Changes
Two themes, both internal-consistency corrections. No activity content, wording
or UUIDs are otherwise changed.
1. Stale activity references synchronized to canonical titles.
Several references named activities by titles that do not exist in the model.
The generator already normalizes
dependsOncomments from the canonical title,so these source comments had drifted from the generated output.
Language and framework specific security rulesandDynamic load of security rules(Guidance) referenced uuid51ebc877as "Basic secure coding rulesfor AI assistants" → corrected to its canonical title "Static load of
security rules".
Dynamic load of security rules(Guidance) andSecurity test generation with AI(Verification) referenced uuid1caad69cas "Security requirements forAI-assisted development" → corrected to its canonical title "Threat
modeling rule".
Threat modeling of AI componentsdescription contained the same stalecross-reference in prose; reworded to reference the Threat modeling rule
activity by its canonical title.
2. Dependency-level progression fixed for one activity.
Static and dynamic analysis of AI generated code(Verification) was level 2but depends on
Static analysis for important server side components(Test andVerification, level 3), so a prerequisite sat one level above the activity that
requires it. Raised the activity to level 3. This also aligns it with its
sibling
No verification bypass for AI generated code(level 3), which sharesthe same level-3 prerequisite.
ARCHITECTURE.mdupdated to match.Files:
ARCHITECTURE.md,src/assets/YAML/default/AgenticAI/Guidance.yaml,src/assets/YAML/default/AgenticAI/Verification.yaml.Compatibility
No activity UUIDs change and no activities are added, removed, moved or renamed,
so assessment continuity is fully preserved. Only
dependsOncomment labels,one prose cross-reference, and one activity's
level(2 → 3) change. Generatedartifacts (
generated/model.yaml,generated/dependency-tree.md) are.gitignored and regenerated/committed by CI, so they are intentionally notpart of this PR.
Validation
Observed results of what was actually run locally:
dependsOncommentsremain, no dependency-level violations remain, no dependency cycles involving
the reviewed Agentic AI activities, all Agentic AI dependencies resolve, all
implementation
$refs used by the Agentic AI dimension resolve, all requiredfields / ranges / tags present,
ARCHITECTURE.mdmatches the canonical YAML(40/40 activities).
dsomm-schema-agentic-ai.json(0 errors).generateDimensions.bash, dockerized): runs without errorsbefore and after; the only meaningful diff in
generated/model.yamlis theintended
level: 2 → 3and the reworded cross-reference, plus theCI-injected version/publisher metadata.
wurstbrot/dsommapp on thegenerated model, received HTTP 200, confirmed the Agentic AI dimension loads
and the activity is served at level 3, with no error entries in the container
log.
--test-urls): not run, because no implementation referencesor URLs are added or changed by this PR.
main, scheduleand manual dispatch, not on
pull_request, so no PR-triggered status checksrun on this branch.
Source basis
The changes are structural/consistency corrections derived from the model's own
rules (canonical titles, the prerequisite semantics of
dependsOndescribed inARCHITECTURE.md) rather than from external standards, so no new references wereadded.
Scope
Intentionally not changed: activity descriptions, risks, measures, assessments,
SAMM/ISO mappings, implementation references, subdimension taxonomy, and every
activity in the other dimensions. Potential consistency observations outside the
Agentic AI dimension were intentionally left untouched to keep this pull request
focused.