Skip to content

Fixing documentation to match new Elements#332

Merged
izar merged 4 commits into
OWASP:masterfrom
izar:master
Jul 5, 2026
Merged

Fixing documentation to match new Elements#332
izar merged 4 commits into
OWASP:masterfrom
izar:master

Conversation

@izar

@izar izar commented May 22, 2026

Copy link
Copy Markdown
Collaborator

Added LLM and Agent, to tm.py and readme.

@colesmj colesmj left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Changes look good. At some point it would be worthwhile revisiting LLM specifically (vs GenAI Model, perhaps) but not a blocker for this PR.

@fkromer

fkromer commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

@izar @colesmj Not sure if this really relates to this MR... just a general note:

In general the risk related to LLM and agent usage heavily depends on where the data processing happens. E.g. if local or self-hosted LLMs are used technologically there is way less risk for all of https://github.com/OWASP/pytm/pull/332/changes#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R552-R565 (despite of hard-coded secret which might end up in git histories and code). The degree of risk remaining depends on the backend/infrastructure used of course.

@izar izar merged commit 9412ef4 into OWASP:master Jul 5, 2026
5 checks passed
@izar

izar commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator Author

@fkromer as much everything, the devil is in the details :D
simply using a local LLM lowers one risk of data leak, for sure. But that assumes programatic use? What if you're using Claude Code with a model off of ollama, but then your skills do the leak for you?
Guess it is time to add more attributes to the new elements!

@fkromer

fkromer commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

@izar Yes, this hint is better placed as a hint in https://github.com/izar/tm_skills I guess.

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.

4 participants