github2gerrit 0.1.7__tar.gz → 0.1.9__tar.gz

{github2gerrit-0.1.7 → github2gerrit-0.1.9}/.pre-commit-config.yaml

@@ -74,11 +74,12 @@ repos:
           - types-PyYAML
           - types-requests
 
-  - repo: https://github.com/RobertCraigie/pyright-python
-    rev: d393df1703a808473b84bd14a2702f4793014031  # frozen: v1.1.404
-    hooks:
-      - id: pyright
-        files: ^(src|scripts|tests)/.+\.py$
+  # Doesn't seem to run reliably inside pre-commit.ci
+  # - repo: https://github.com/RobertCraigie/pyright-python
+  #   rev: d393df1703a808473b84bd14a2702f4793014031  # frozen: v1.1.404
+  #   hooks:
+  #     - id: pyright
+  #       files: ^(src|scripts|tests)/.+\.py$
 
   - repo: https://github.com/btford/write-good
     rev: ab66ce10136dfad5146e69e70f82a3efac8842c1 # frozen: v1.0.8

{github2gerrit-0.1.7 → github2gerrit-0.1.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: github2gerrit
-Version: 0.1.7
+Version: 0.1.9
 Summary: Submit a GitHub pull request to a Gerrit repository.
 Author-email: Matthew Watkins <mwatkins@linuxfoundation.org>
 License-Expression: Apache-2.0
@@ -297,7 +297,7 @@ jobs:
     steps:
       - name: Submit PR to Gerrit
         id: g2g
-        uses: lfit/github2gerrit@main
+        uses: lfreleng-actions/github2gerrit-action@main
         with:
           SUBMIT_SINGLE_COMMITS: "false"
           USE_PR_AS_COMMIT: "false"
@@ -343,7 +343,7 @@ uv run github2gerrit
 uvx github2gerrit https://github.com/owner/repo/pull/123
 
 # Install from specific version/source
-uvx --from git+https://github.com/lfit/github2gerrit@main github2gerrit https://github.com/owner/repo/pull/123
+uvx --from git+https://github.com/lfreleng-actions/github2gerrit-action@main github2gerrit https://github.com/owner/repo/pull/123
 ```
 
 ### Available Options
@@ -524,7 +524,7 @@ jobs:
 
       - name: Submit PR to Gerrit (with explicit overrides)
         id: g2g
-        uses: lfit/github2gerrit@main
+        uses: lfreleng-actions/github2gerrit-action@main
         with:
           # Behavior
           SUBMIT_SINGLE_COMMITS: "false"
@@ -777,7 +777,7 @@ For CI/CD pipelines (like GitHub Actions), use `uvx` to install and run without
 uvx github2gerrit <PR_URL> --dry-run
 
 # Install from a specific version or source
-uvx --from git+https://github.com/lfit/github2gerrit@main github2gerrit <PR_URL>
+uvx --from git+https://github.com/lfreleng-actions/github2gerrit-action@main github2gerrit <PR_URL>
 
 # Run with specific Python version
 uvx --python 3.11 github2gerrit <PR_URL>

{github2gerrit-0.1.7 → github2gerrit-0.1.9}/README.md

@@ -257,7 +257,7 @@ jobs:
     steps:
       - name: Submit PR to Gerrit
         id: g2g
-        uses: lfit/github2gerrit@main
+        uses: lfreleng-actions/github2gerrit-action@main
         with:
           SUBMIT_SINGLE_COMMITS: "false"
           USE_PR_AS_COMMIT: "false"
@@ -303,7 +303,7 @@ uv run github2gerrit
 uvx github2gerrit https://github.com/owner/repo/pull/123
 
 # Install from specific version/source
-uvx --from git+https://github.com/lfit/github2gerrit@main github2gerrit https://github.com/owner/repo/pull/123
+uvx --from git+https://github.com/lfreleng-actions/github2gerrit-action@main github2gerrit https://github.com/owner/repo/pull/123
 ```
 
 ### Available Options
@@ -484,7 +484,7 @@ jobs:
 
       - name: Submit PR to Gerrit (with explicit overrides)
         id: g2g
-        uses: lfit/github2gerrit@main
+        uses: lfreleng-actions/github2gerrit-action@main
         with:
           # Behavior
           SUBMIT_SINGLE_COMMITS: "false"
@@ -737,7 +737,7 @@ For CI/CD pipelines (like GitHub Actions), use `uvx` to install and run without
 uvx github2gerrit <PR_URL> --dry-run
 
 # Install from a specific version or source
-uvx --from git+https://github.com/lfit/github2gerrit@main github2gerrit <PR_URL>
+uvx --from git+https://github.com/lfreleng-actions/github2gerrit-action@main github2gerrit <PR_URL>
 
 # Run with specific Python version
 uvx --python 3.11 github2gerrit <PR_URL>

{github2gerrit-0.1.7 → github2gerrit-0.1.9}/REUSE.toml

@@ -7,6 +7,7 @@ version = 1
 
 [[annotations]]
 path = [
+         "docs/*.png",
 	 "*.lock"
 ]
 SPDX-License-Identifier = "Apache-2.0"

{github2gerrit-0.1.7 → github2gerrit-0.1.9}/src/github2gerrit/cli.py

@@ -359,15 +359,21 @@ def main(
     # URL mode handling
     if target_url:
         org, repo, pr = _parse_github_target(target_url)
+        log.debug("Parsed GitHub URL: org=%s, repo=%s, pr=%s", org, repo, pr)
         if org:
             os.environ["ORGANIZATION"] = org
+            log.debug("Set ORGANIZATION=%s", org)
         if org and repo:
-            os.environ["GITHUB_REPOSITORY"] = f"{org}/{repo}"
+            github_repo = f"{org}/{repo}"
+            os.environ["GITHUB_REPOSITORY"] = github_repo
+            log.debug("Set GITHUB_REPOSITORY=%s", github_repo)
         if pr:
             os.environ["PR_NUMBER"] = str(pr)
             os.environ["SYNC_ALL_OPEN_PRS"] = "false"
+            log.debug("Set PR_NUMBER=%s", pr)
         else:
             os.environ["SYNC_ALL_OPEN_PRS"] = "true"
+            log.debug("Set SYNC_ALL_OPEN_PRS=true")
         os.environ["G2G_TARGET_URL"] = "1"
     # Debug: Show environment at CLI startup
     log.debug("CLI startup environment check:")

{github2gerrit-0.1.7 → github2gerrit-0.1.9}/src/github2gerrit/config.py

@@ -131,6 +131,22 @@ def _coerce_value(raw: str) -> str:
     # like SSH keys or known_hosts entries can be specified inline using
     # '\n' or '\r\n' in configuration files.
     normalized_newlines = unquoted.replace("\\r\\n", "\n").replace("\\n", "\n").replace("\r\n", "\n")
+
+    # Additional sanitization for SSH private keys
+    if ("-----BEGIN" in normalized_newlines and "PRIVATE KEY-----" in normalized_newlines) or (
+        "ssh-" in normalized_newlines.lower() and "key" in normalized_newlines.lower()
+    ):
+        # Clean up SSH key formatting: remove extra whitespace, normalize line endings
+        lines = normalized_newlines.split("\n")
+        sanitized_lines = []
+        for line in lines:
+            cleaned = line.strip()
+            if cleaned:
+                # Remove any stray quotes that might have been embedded in the key content
+                cleaned = cleaned.replace('"', "").replace("'", "")
+                sanitized_lines.append(cleaned)
+        normalized_newlines = "\n".join(sanitized_lines)
+
     b = _normalize_bool_like(normalized_newlines)
     return b if b is not None else normalized_newlines
 
@@ -149,7 +165,7 @@ def _select_section(
 
 def _load_ini(path: Path) -> configparser.RawConfigParser:
     cp = configparser.RawConfigParser()
-    # Preserve option case; mypy requires a cast for attribute assignment
+    # Preserve option case; mypy requires a cast for attribute requirement
     cast(Any, cp).optionxform = str
     try:
         with path.open("r", encoding="utf-8") as fh:
@@ -162,6 +178,9 @@ def _load_ini(path: Path) -> configparser.RawConfigParser:
         # We collapse these into a single line with '\n' escapes so that
         # configparser can ingest them reliably; later, _coerce_value()
         # converts the escapes back to real newlines.
+        #
+        # We also handle SSH private keys and other multi-line values that
+        # might have formatting inconsistencies by sanitizing them.
         lines = raw_text.splitlines()
         out_lines: list[str] = []
         i = 0
@@ -171,6 +190,8 @@ def _load_ini(path: Path) -> configparser.RawConfigParser:
             if eq_idx != -1:
                 left = line[: eq_idx + 1]
                 rhs = line[eq_idx + 1 :].strip()
+
+                # Handle standard multi-line quoted values: key = "
                 if rhs == '"':
                     i += 1
                     block: list[str] = []
@@ -187,10 +208,64 @@ def _load_ini(path: Path) -> configparser.RawConfigParser:
                     else:
                         # No closing quote found; fall through
                         # and keep original line
+                        log.debug("Multi-line quote not properly closed for line: %s", line[:50])
                         out_lines.append(line)
                         continue
+
+                # Handle SSH private keys and other values that start with a quote
+                # but contain embedded content that might confuse configparser
+                elif rhs.startswith('"') and not rhs.endswith('"'):
+                    # This looks like a multi-line value that starts on the same line
+                    # Collect all content until we find a line ending with a quote
+                    content_lines = [rhs[1:]]  # Remove opening quote
+                    i += 1
+
+                    while i < len(lines):
+                        current_line = lines[i]
+                        if current_line.strip().endswith('"') and not current_line.strip().endswith('\\"'):
+                            # Found closing quote - remove it and add final line
+                            final_content = current_line.rstrip()
+                            if final_content.endswith('"'):
+                                final_content = final_content[:-1]
+                            if final_content:  # Only add if there's content after removing quote
+                                content_lines.append(final_content)
+                            break
+                        else:
+                            content_lines.append(current_line)
+                        i += 1
+
+                    # Join all content and sanitize for SSH keys
+                    full_content = "\\n".join(content_lines)
+
+                    # Special handling for SSH private keys - remove extra whitespace and line breaks
+                    key_name = left.split("=")[0].strip().upper()
+                    if "SSH" in key_name and "KEY" in key_name:
+                        # For SSH keys, clean up base64 content by removing whitespace within lines
+                        sanitized_lines = []
+                        for content_line in content_lines:
+                            cleaned = content_line.strip()
+                            # Preserve SSH key headers/footers but clean base64 content
+                            if cleaned.startswith("-----") or not cleaned:
+                                sanitized_lines.append(cleaned)
+                            else:
+                                # Remove any embedded quotes and whitespace from base64 content
+                                cleaned = cleaned.replace('"', "").replace("'", "").strip()
+                                if cleaned:
+                                    sanitized_lines.append(cleaned)
+                        full_content = "\\n".join(sanitized_lines)
+
+                    log.debug(
+                        "Processed multi-line value for key %s (length: %d)",
+                        left.split("=")[0].strip(),
+                        len(full_content),
+                    )
+                    out_lines.append(f'{left} "{full_content}"')
+                    i += 1
+                    continue
+
             out_lines.append(line)
             i += 1
+
         preprocessed = "\n".join(out_lines) + ("\n" if out_lines else "")
         cp.read_string(preprocessed)
     except FileNotFoundError:
@@ -272,6 +347,8 @@ def load_org_config(
 
     # Report unknown configuration keys to help users catch typos
     unknown_keys = set(normalized.keys()) - KNOWN_KEYS
+    log.debug("All parsed keys from config: %s", sorted(normalized.keys()))
+    log.debug("Known keys: %s", sorted(KNOWN_KEYS))
     if unknown_keys:
         log.warning(
             "Unknown configuration keys found in [%s]: %s. "

{github2gerrit-0.1.7 → github2gerrit-0.1.9}/src/github2gerrit/github_api.py

@@ -171,7 +171,9 @@ def build_client(token: str | None = None) -> GhClient:
 def get_repo_from_env(client: GhClient) -> GhRepository:
     """Return the repository object based on GITHUB_REPOSITORY."""
     full = _getenv_str("GITHUB_REPOSITORY")
+    log.debug("GITHUB_REPOSITORY environment variable: '%s'", full)
     if not full or "/" not in full:
+        log.error("Invalid GITHUB_REPOSITORY: '%s' (expected format: 'owner/repo')", full)
         raise ValueError(_MSG_BAD_GITHUB_REPOSITORY)
     repo = client.get_repo(full)
     return repo

{github2gerrit-0.1.7 → github2gerrit-0.1.9}/src/github2gerrit.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: github2gerrit
-Version: 0.1.7
+Version: 0.1.9
 Summary: Submit a GitHub pull request to a Gerrit repository.
 Author-email: Matthew Watkins <mwatkins@linuxfoundation.org>
 License-Expression: Apache-2.0
@@ -297,7 +297,7 @@ jobs:
     steps:
       - name: Submit PR to Gerrit
         id: g2g
-        uses: lfit/github2gerrit@main
+        uses: lfreleng-actions/github2gerrit-action@main
         with:
           SUBMIT_SINGLE_COMMITS: "false"
           USE_PR_AS_COMMIT: "false"
@@ -343,7 +343,7 @@ uv run github2gerrit
 uvx github2gerrit https://github.com/owner/repo/pull/123
 
 # Install from specific version/source
-uvx --from git+https://github.com/lfit/github2gerrit@main github2gerrit https://github.com/owner/repo/pull/123
+uvx --from git+https://github.com/lfreleng-actions/github2gerrit-action@main github2gerrit https://github.com/owner/repo/pull/123
 ```
 
 ### Available Options
@@ -524,7 +524,7 @@ jobs:
 
       - name: Submit PR to Gerrit (with explicit overrides)
         id: g2g
-        uses: lfit/github2gerrit@main
+        uses: lfreleng-actions/github2gerrit-action@main
         with:
           # Behavior
           SUBMIT_SINGLE_COMMITS: "false"
@@ -777,7 +777,7 @@ For CI/CD pipelines (like GitHub Actions), use `uvx` to install and run without
 uvx github2gerrit <PR_URL> --dry-run
 
 # Install from a specific version or source
-uvx --from git+https://github.com/lfit/github2gerrit@main github2gerrit <PR_URL>
+uvx --from git+https://github.com/lfreleng-actions/github2gerrit-action@main github2gerrit <PR_URL>
 
 # Run with specific Python version
 uvx --python 3.11 github2gerrit <PR_URL>

{github2gerrit-0.1.7 → github2gerrit-0.1.9}/src/github2gerrit.egg-info/SOURCES.txt

@@ -24,6 +24,7 @@ uv.lock
 .github/workflows/sha-pinned-actions.yaml
 .github/workflows/testing.yaml
 LICENSES/Apache-2.0.txt
+docs/github2gerrit_token_permissions_classic.png
 src/github2gerrit/__init__.py
 src/github2gerrit/cli.py
 src/github2gerrit/commit_normalization.py