Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions 4 pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ build-backend = "hatchling.build"

[project]
name = "socketsecurity"
version = "2.2.85"
version = "2.2.86"
requires-python = ">= 3.11"
license = {"file" = "LICENSE"}
dependencies = [
Expand All @@ -16,7 +16,7 @@ dependencies = [
'GitPython',
'packaging',
'python-dotenv',
"socketdev>=3.0.32,<4.0.0",
"socketdev>=3.0.33,<4.0.0",
"bs4>=0.0.2",
"markdown>=3.10",
]
Expand Down
2 changes: 1 addition & 1 deletion 2 socketsecurity/__init__.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
__author__ = 'socket.dev'
__version__ = '2.2.85'
__version__ = '2.2.86'
USER_AGENT = f'SocketPythonCLI/{__version__}'
6 changes: 4 additions & 2 deletions 6 socketsecurity/core/classes.py
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,7 @@ def from_diff_artifact(cls, data: dict) -> "Package":
name=data["name"],
version=data["version"],
type=data["type"],
score=data.get("score", data.get("scores", {})),
score=data.get("score") or data.get("scores") or {},
alerts=data.get("alerts", []),
author=data.get("author", []),
size=data.get("size"),
Expand Down Expand Up @@ -236,7 +236,7 @@ def from_diff_artifact(cls, data: dict) -> "Package":
name=data["name"],
version=data["version"],
type=data["type"],
score=data.get("score", data.get("scores", {})),
score=data.get("score") or data.get("scores") or {},
alerts=data.get("alerts", []),
author=data.get("author", []),
size=data.get("size"),
Expand Down Expand Up @@ -448,6 +448,8 @@ def __init__(self, **kwargs):
self.capabilities = []
if not hasattr(self, "is_new"):
self.is_new = False
if not hasattr(self, "scores") or self.scores is None:
self.scores = {}
self.author_url = Purl.generate_author_data(self.author, self.ecosystem)

@staticmethod
Expand Down
25 changes: 20 additions & 5 deletions 25 socketsecurity/core/messages.py
Original file line number Diff line number Diff line change
Expand Up @@ -1179,12 +1179,27 @@ def score_to_badge(score):
score_percent = int(score * 100) # Convert to integer percentage
return f"[![{score_percent}](https://github-app-statics.socket.dev/score-{score_percent}.svg)]({added.url})"

def get_score_for_badge(score_name: str) -> float:
scores = getattr(added, "scores", None)
if isinstance(scores, dict):
raw_score = scores.get(score_name)
else:
raw_score = getattr(scores, score_name, None) if scores is not None else None

if raw_score is None:
return 1.0

score = float(raw_score)
if score > 1:
score = score / 100
return max(0.0, min(score, 1.0))

# Generate badges for each score type
supply_chain_risk_badge = score_to_badge(added.scores.get("supplyChain", 100))
vulnerability_badge = score_to_badge(added.scores.get("vulnerability", 100))
quality_badge = score_to_badge(added.scores.get("quality", 100))
maintenance_badge = score_to_badge(added.scores.get("maintenance", 100))
license_badge = score_to_badge(added.scores.get("license", 100))
supply_chain_risk_badge = score_to_badge(get_score_for_badge("supplyChain"))
vulnerability_badge = score_to_badge(get_score_for_badge("vulnerability"))
quality_badge = score_to_badge(get_score_for_badge("quality"))
maintenance_badge = score_to_badge(get_score_for_badge("maintenance"))
license_badge = score_to_badge(get_score_for_badge("license"))

# Add the row for this package
row = [
Expand Down
67 changes: 67 additions & 0 deletions 67 tests/unit/test_dependency_overview.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,67 @@
from socketsecurity.core.classes import Diff, Package, Purl
from socketsecurity.core.messages import Messages


def _make_purl(name: str, scores) -> Purl:
return Purl(
id=f"pkg:npm/{name}@1.0.0",
name=name,
version="1.0.0",
ecosystem="npm",
direct=True,
introduced_by=[("direct", "package.json")],
author=["test-author"],
size=1000,
transitives=0,
url=f"https://socket.dev/npm/package/{name}/overview/1.0.0",
purl=f"pkg:npm/{name}@1.0.0",
scores=scores,
)


def test_package_from_diff_artifact_normalizes_null_score():
package = Package.from_diff_artifact(
{
"id": "pkg:npm/example@1.0.0",
"name": "example",
"version": "1.0.0",
"type": "npm",
"diffType": "added",
"score": None,
"alerts": [],
"author": [],
"topLevelAncestors": [],
"direct": True,
"manifestFiles": [],
}
)

assert package.score == {}


def test_dependency_overview_template_defaults_missing_or_null_scores(tmp_path, monkeypatch):
monkeypatch.chdir(tmp_path)

diff = Diff(
id="test-diff",
diff_url="https://socket.dev/test-diff",
new_packages=[
_make_purl("missing-scores", None),
_make_purl(
"partial-scores",
{
"supplyChain": 0.42,
"vulnerability": None,
},
),
],
removed_packages=[],
new_alerts=[],
)

comment = Messages.dependency_overview_template(diff)

assert "Socket Security: Dependency Overview" in comment
assert "score-42.svg" in comment
assert "score-100.svg" in comment
assert "score-10000.svg" not in comment
10 changes: 5 additions & 5 deletions 10 uv.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading
Morty Proxy This is a proxified and sanitized view of the page, visit original site.