@codyswann/lisa 4.55.9 → 4.55.11
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/core/nightly-e2e-guard-behavior-certificate.js +2 -2
- package/dist/core/nightly-e2e-guard-behavior-certificate.js.map +1 -1
- package/dist/core/upstream-evidence-manifest.d.ts.map +1 -1
- package/dist/core/upstream-evidence-manifest.js +14 -3
- package/dist/core/upstream-evidence-manifest.js.map +1 -1
- package/package.json +4 -4
- package/plugins/lisa/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa/rules/eager/00-rule-index.md +1 -0
- package/plugins/lisa/rules/eager/never-name-a-downstream-project.md +9 -0
- package/plugins/lisa/rules/reference/never-name-a-downstream-project.md +37 -0
- package/plugins/lisa-agy/plugin.json +1 -1
- package/plugins/lisa-cdk/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk-agy/plugin.json +1 -1
- package/plugins/lisa-cdk-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-copilot/rules/eager/00-rule-index.md +1 -0
- package/plugins/lisa-copilot/rules/eager/never-name-a-downstream-project.md +9 -0
- package/plugins/lisa-copilot/rules/reference/never-name-a-downstream-project.md +37 -0
- package/plugins/lisa-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cursor/rules/00-rule-index.mdc +1 -0
- package/plugins/lisa-cursor/rules/never-name-a-downstream-project-reference.mdc +42 -0
- package/plugins/lisa-cursor/rules/never-name-a-downstream-project.mdc +14 -0
- package/plugins/lisa-expo/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-expo/.codex-plugin/skills/jira-journey/SKILL.md +7 -2
- package/plugins/lisa-expo/.codex-plugin/skills/jira-journey/scripts/parse-plan.py +32 -0
- package/plugins/lisa-expo/skills/jira-journey/SKILL.md +7 -2
- package/plugins/lisa-expo/skills/jira-journey/scripts/parse-plan.py +32 -0
- package/plugins/lisa-expo-agy/plugin.json +1 -1
- package/plugins/lisa-expo-agy/skills/jira-journey/SKILL.md +7 -2
- package/plugins/lisa-expo-agy/skills/jira-journey/scripts/parse-plan.py +32 -0
- package/plugins/lisa-expo-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo-copilot/skills/jira-journey/SKILL.md +7 -2
- package/plugins/lisa-expo-copilot/skills/jira-journey/scripts/parse-plan.py +32 -0
- package/plugins/lisa-expo-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo-cursor/skills/jira-journey/SKILL.md +7 -2
- package/plugins/lisa-expo-cursor/skills/jira-journey/scripts/parse-plan.py +32 -0
- package/plugins/lisa-harper-fabric/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-agy/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs-agy/plugin.json +1 -1
- package/plugins/lisa-nestjs-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw-agy/plugin.json +1 -1
- package/plugins/lisa-openclaw-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser-agy/plugin.json +1 -1
- package/plugins/lisa-phaser-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-rails-agy/plugin.json +1 -1
- package/plugins/lisa-rails-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript-agy/plugin.json +1 -1
- package/plugins/lisa-typescript-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki-agy/plugin.json +1 -1
- package/plugins/lisa-wiki-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/src/base/rules/eager/00-rule-index.md +1 -0
- package/plugins/src/base/rules/eager/never-name-a-downstream-project.md +9 -0
- package/plugins/src/base/rules/reference/never-name-a-downstream-project.md +37 -0
- package/plugins/src/expo/skills/jira-journey/SKILL.md +7 -2
- package/plugins/src/expo/skills/jira-journey/scripts/parse-plan.py +32 -0
package/package.json
CHANGED
|
@@ -187,7 +187,7 @@
|
|
|
187
187
|
"zod-validation-error": "^4.0.0"
|
|
188
188
|
},
|
|
189
189
|
"name": "@codyswann/lisa",
|
|
190
|
-
"version": "4.55.
|
|
190
|
+
"version": "4.55.11",
|
|
191
191
|
"description": "Claude Code governance framework that applies guardrails, guidance, and automated enforcement to projects",
|
|
192
192
|
"main": "dist/index.js",
|
|
193
193
|
"exports": {
|
|
@@ -336,7 +336,7 @@
|
|
|
336
336
|
"test": "tests"
|
|
337
337
|
},
|
|
338
338
|
"types": "./dist/index.d.ts",
|
|
339
|
-
"lisaReleaseCommit": "
|
|
340
|
-
"gitHead": "
|
|
341
|
-
"lisaReleaseTag": "v4.55.
|
|
339
|
+
"lisaReleaseCommit": "4936a1c0995d3406d00b906b4390ffe2e7f03c59",
|
|
340
|
+
"gitHead": "4936a1c0995d3406d00b906b4390ffe2e7f03c59",
|
|
341
|
+
"lisaReleaseTag": "v4.55.11"
|
|
342
342
|
}
|
|
@@ -76,6 +76,7 @@ Paths are relative to this file.
|
|
|
76
76
|
| `documentation-source-paths` | moving, absorbing, or deleting a docs-like directory | [../reference/documentation-source-paths.md](../reference/documentation-source-paths.md) |
|
|
77
77
|
| `project-learnings` | you need the learnings ledger's entry schema and field semantics | [../reference/project-learnings.md](../reference/project-learnings.md) |
|
|
78
78
|
| `factory-model` | you need the shape of the factory model, not its obligations | [../reference/factory-model.md](../reference/factory-model.md) |
|
|
79
|
+
| `never-name-a-downstream-project` | you are about to refer to a downstream project in writing | [../reference/never-name-a-downstream-project.md](../reference/never-name-a-downstream-project.md) |
|
|
79
80
|
|
|
80
81
|
The obligations that follow from `factory-model` are in `base-rules`, which is
|
|
81
82
|
eager. This entry is the exposition behind them.
|
|
@@ -0,0 +1,9 @@
|
|
|
1
|
+
# Never Name a Downstream Project
|
|
2
|
+
|
|
3
|
+
Refer to a downstream project by its **role in the argument** — never by organization, product, or repository name — in tracker items, pull requests, commits, rules, skills, anything that outlives the session.
|
|
4
|
+
|
|
5
|
+
Write the evidence, not the identity: "a caller repo", "repo A" / "repo B" when two are contrasted.
|
|
6
|
+
|
|
7
|
+
Default-deny: it covers names nobody listed. `block-host-name-leak` catches some of them and is a rate reduction, never a proof.
|
|
8
|
+
|
|
9
|
+
Full prose: [reference/never-name-a-downstream-project.md](../reference/never-name-a-downstream-project.md).
|
|
@@ -0,0 +1,37 @@
|
|
|
1
|
+
# Never Name a Downstream Project
|
|
2
|
+
|
|
3
|
+
## The rule
|
|
4
|
+
|
|
5
|
+
A downstream project's identity — its organization, product, or repository name — does not belong in anything that outlives the session it was written in: tracker items (titles, descriptions, comments), pull request titles and bodies, commit messages, wiki and ingested sources, and the text of rules and skills.
|
|
6
|
+
|
|
7
|
+
## Why the identity is never the point
|
|
8
|
+
|
|
9
|
+
An observation earns its place by being reproducible, not by its provenance. "A caller repo pinned all 64 of its call sites" and the same sentence with a name carry exactly the same engineering weight — the second just adds an obligation for whoever reads it later.
|
|
10
|
+
|
|
11
|
+
Where an argument appears to need the name, that is a signal the argument is under-specified. The fix is to state the condition that made it true — the version, the config, the shape of the tree — not to identify the place it was observed.
|
|
12
|
+
|
|
13
|
+
## How to write it instead
|
|
14
|
+
|
|
15
|
+
| situation | write |
|
|
16
|
+
|---|---|
|
|
17
|
+
| one project | "a caller repo", "a downstream project", "the reference repository" |
|
|
18
|
+
| two contrasted | "repo A" / "repo B" |
|
|
19
|
+
| several that must stay distinct in one sentence | `<project-1>`, `<project-2>` |
|
|
20
|
+
| a real identifier | `<ticket>` |
|
|
21
|
+
|
|
22
|
+
Rewrite the prose rather than substituting tokens wherever a placeholder reads badly: "follow `<name>`'s serverless pattern" becomes "follow the reference serverless pattern", not "follow `<project-1>`'s serverless pattern".
|
|
23
|
+
|
|
24
|
+
## Enforcement is partial, and must not be relied on
|
|
25
|
+
|
|
26
|
+
`block-host-name-leak` compares against a **curated denylist** and inspects **Bash tool calls only**. If the detector cannot be loaded, the hook warns and permits the write without inspecting the text. These limits follow:
|
|
27
|
+
|
|
28
|
+
- a name nobody has added to the list is invisible to it;
|
|
29
|
+
- a write issued by a script that reads its own body — rather than by a CLI given a `--body-file` — never reaches it at all.
|
|
30
|
+
|
|
31
|
+
When the detector is available, a clean run means "no listed name appeared", never "this text is safe". When it is unavailable, no inspection occurred. This rule is default-deny and holds independently of whether the guard fires.
|
|
32
|
+
|
|
33
|
+
Note also that the denylist cannot be the primary control: this repository is public and `dist/` ships to npm, so the entries most worth catching are the ones least safe to enumerate there.
|
|
34
|
+
|
|
35
|
+
## Prior reference is not licence
|
|
36
|
+
|
|
37
|
+
An existing ticket, brief, or ingested source that already carries a name does not authorise repeating it, and inherited text is the most common way such names spread — ingestion carries them forward, and each new document that quotes the last one multiplies them. When you meet one, replace it in what you are writing; do not propagate it because it was already there.
|
|
@@ -76,6 +76,7 @@ Paths are relative to this file.
|
|
|
76
76
|
| `documentation-source-paths` | moving, absorbing, or deleting a docs-like directory | [../reference/documentation-source-paths.md](../reference/documentation-source-paths.md) |
|
|
77
77
|
| `project-learnings` | you need the learnings ledger's entry schema and field semantics | [../reference/project-learnings.md](../reference/project-learnings.md) |
|
|
78
78
|
| `factory-model` | you need the shape of the factory model, not its obligations | [../reference/factory-model.md](../reference/factory-model.md) |
|
|
79
|
+
| `never-name-a-downstream-project` | you are about to refer to a downstream project in writing | [../reference/never-name-a-downstream-project.md](../reference/never-name-a-downstream-project.md) |
|
|
79
80
|
|
|
80
81
|
The obligations that follow from `factory-model` are in `base-rules`, which is
|
|
81
82
|
eager. This entry is the exposition behind them.
|
|
@@ -0,0 +1,9 @@
|
|
|
1
|
+
# Never Name a Downstream Project
|
|
2
|
+
|
|
3
|
+
Refer to a downstream project by its **role in the argument** — never by organization, product, or repository name — in tracker items, pull requests, commits, rules, skills, anything that outlives the session.
|
|
4
|
+
|
|
5
|
+
Write the evidence, not the identity: "a caller repo", "repo A" / "repo B" when two are contrasted.
|
|
6
|
+
|
|
7
|
+
Default-deny: it covers names nobody listed. `block-host-name-leak` catches some of them and is a rate reduction, never a proof.
|
|
8
|
+
|
|
9
|
+
Full prose: [reference/never-name-a-downstream-project.md](../reference/never-name-a-downstream-project.md).
|
|
@@ -0,0 +1,37 @@
|
|
|
1
|
+
# Never Name a Downstream Project
|
|
2
|
+
|
|
3
|
+
## The rule
|
|
4
|
+
|
|
5
|
+
A downstream project's identity — its organization, product, or repository name — does not belong in anything that outlives the session it was written in: tracker items (titles, descriptions, comments), pull request titles and bodies, commit messages, wiki and ingested sources, and the text of rules and skills.
|
|
6
|
+
|
|
7
|
+
## Why the identity is never the point
|
|
8
|
+
|
|
9
|
+
An observation earns its place by being reproducible, not by its provenance. "A caller repo pinned all 64 of its call sites" and the same sentence with a name carry exactly the same engineering weight — the second just adds an obligation for whoever reads it later.
|
|
10
|
+
|
|
11
|
+
Where an argument appears to need the name, that is a signal the argument is under-specified. The fix is to state the condition that made it true — the version, the config, the shape of the tree — not to identify the place it was observed.
|
|
12
|
+
|
|
13
|
+
## How to write it instead
|
|
14
|
+
|
|
15
|
+
| situation | write |
|
|
16
|
+
|---|---|
|
|
17
|
+
| one project | "a caller repo", "a downstream project", "the reference repository" |
|
|
18
|
+
| two contrasted | "repo A" / "repo B" |
|
|
19
|
+
| several that must stay distinct in one sentence | `<project-1>`, `<project-2>` |
|
|
20
|
+
| a real identifier | `<ticket>` |
|
|
21
|
+
|
|
22
|
+
Rewrite the prose rather than substituting tokens wherever a placeholder reads badly: "follow `<name>`'s serverless pattern" becomes "follow the reference serverless pattern", not "follow `<project-1>`'s serverless pattern".
|
|
23
|
+
|
|
24
|
+
## Enforcement is partial, and must not be relied on
|
|
25
|
+
|
|
26
|
+
`block-host-name-leak` compares against a **curated denylist** and inspects **Bash tool calls only**. If the detector cannot be loaded, the hook warns and permits the write without inspecting the text. These limits follow:
|
|
27
|
+
|
|
28
|
+
- a name nobody has added to the list is invisible to it;
|
|
29
|
+
- a write issued by a script that reads its own body — rather than by a CLI given a `--body-file` — never reaches it at all.
|
|
30
|
+
|
|
31
|
+
When the detector is available, a clean run means "no listed name appeared", never "this text is safe". When it is unavailable, no inspection occurred. This rule is default-deny and holds independently of whether the guard fires.
|
|
32
|
+
|
|
33
|
+
Note also that the denylist cannot be the primary control: this repository is public and `dist/` ships to npm, so the entries most worth catching are the ones least safe to enumerate there.
|
|
34
|
+
|
|
35
|
+
## Prior reference is not licence
|
|
36
|
+
|
|
37
|
+
An existing ticket, brief, or ingested source that already carries a name does not authorise repeating it, and inherited text is the most common way such names spread — ingestion carries them forward, and each new document that quotes the last one multiplies them. When you meet one, replace it in what you are writing; do not propagate it because it was already there.
|
|
@@ -81,6 +81,7 @@ Paths are relative to this file.
|
|
|
81
81
|
| `documentation-source-paths` | moving, absorbing, or deleting a docs-like directory | [../reference/documentation-source-paths.md](documentation-source-paths-reference.mdc) |
|
|
82
82
|
| `project-learnings` | you need the learnings ledger's entry schema and field semantics | [../reference/project-learnings.md](project-learnings-reference.mdc) |
|
|
83
83
|
| `factory-model` | you need the shape of the factory model, not its obligations | [../reference/factory-model.md](factory-model-reference.mdc) |
|
|
84
|
+
| `never-name-a-downstream-project` | you are about to refer to a downstream project in writing | [../reference/never-name-a-downstream-project.md](never-name-a-downstream-project-reference.mdc) |
|
|
84
85
|
|
|
85
86
|
The obligations that follow from `factory-model` are in `base-rules`, which is
|
|
86
87
|
eager. This entry is the exposition behind them.
|
|
@@ -0,0 +1,42 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: "Never Name a Downstream Project"
|
|
3
|
+
alwaysApply: false
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Never Name a Downstream Project
|
|
7
|
+
|
|
8
|
+
## The rule
|
|
9
|
+
|
|
10
|
+
A downstream project's identity — its organization, product, or repository name — does not belong in anything that outlives the session it was written in: tracker items (titles, descriptions, comments), pull request titles and bodies, commit messages, wiki and ingested sources, and the text of rules and skills.
|
|
11
|
+
|
|
12
|
+
## Why the identity is never the point
|
|
13
|
+
|
|
14
|
+
An observation earns its place by being reproducible, not by its provenance. "A caller repo pinned all 64 of its call sites" and the same sentence with a name carry exactly the same engineering weight — the second just adds an obligation for whoever reads it later.
|
|
15
|
+
|
|
16
|
+
Where an argument appears to need the name, that is a signal the argument is under-specified. The fix is to state the condition that made it true — the version, the config, the shape of the tree — not to identify the place it was observed.
|
|
17
|
+
|
|
18
|
+
## How to write it instead
|
|
19
|
+
|
|
20
|
+
| situation | write |
|
|
21
|
+
|---|---|
|
|
22
|
+
| one project | "a caller repo", "a downstream project", "the reference repository" |
|
|
23
|
+
| two contrasted | "repo A" / "repo B" |
|
|
24
|
+
| several that must stay distinct in one sentence | `<project-1>`, `<project-2>` |
|
|
25
|
+
| a real identifier | `<ticket>` |
|
|
26
|
+
|
|
27
|
+
Rewrite the prose rather than substituting tokens wherever a placeholder reads badly: "follow `<name>`'s serverless pattern" becomes "follow the reference serverless pattern", not "follow `<project-1>`'s serverless pattern".
|
|
28
|
+
|
|
29
|
+
## Enforcement is partial, and must not be relied on
|
|
30
|
+
|
|
31
|
+
`block-host-name-leak` compares against a **curated denylist** and inspects **Bash tool calls only**. If the detector cannot be loaded, the hook warns and permits the write without inspecting the text. These limits follow:
|
|
32
|
+
|
|
33
|
+
- a name nobody has added to the list is invisible to it;
|
|
34
|
+
- a write issued by a script that reads its own body — rather than by a CLI given a `--body-file` — never reaches it at all.
|
|
35
|
+
|
|
36
|
+
When the detector is available, a clean run means "no listed name appeared", never "this text is safe". When it is unavailable, no inspection occurred. This rule is default-deny and holds independently of whether the guard fires.
|
|
37
|
+
|
|
38
|
+
Note also that the denylist cannot be the primary control: this repository is public and `dist/` ships to npm, so the entries most worth catching are the ones least safe to enumerate there.
|
|
39
|
+
|
|
40
|
+
## Prior reference is not licence
|
|
41
|
+
|
|
42
|
+
An existing ticket, brief, or ingested source that already carries a name does not authorise repeating it, and inherited text is the most common way such names spread — ingestion carries them forward, and each new document that quotes the last one multiplies them. When you meet one, replace it in what you are writing; do not propagate it because it was already there.
|
|
@@ -0,0 +1,14 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: "Never Name a Downstream Project"
|
|
3
|
+
alwaysApply: true
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Never Name a Downstream Project
|
|
7
|
+
|
|
8
|
+
Refer to a downstream project by its **role in the argument** — never by organization, product, or repository name — in tracker items, pull requests, commits, rules, skills, anything that outlives the session.
|
|
9
|
+
|
|
10
|
+
Write the evidence, not the identity: "a caller repo", "repo A" / "repo B" when two are contrasted.
|
|
11
|
+
|
|
12
|
+
Default-deny: it covers names nobody listed. `block-host-name-leak` catches some of them and is a rate reduction, never a proof.
|
|
13
|
+
|
|
14
|
+
Full prose: [reference/never-name-a-downstream-project.md](never-name-a-downstream-project-reference.mdc).
|
|
@@ -17,8 +17,13 @@ Read a JIRA ticket's Validation Journey, execute it via Playwright MCP browser t
|
|
|
17
17
|
## Prerequisites
|
|
18
18
|
|
|
19
19
|
- `JIRA_API_TOKEN` environment variable set
|
|
20
|
-
- `jira-cli` configured
|
|
21
|
-
|
|
20
|
+
- `jira-cli` configured in the operator's `~/.config/.jira/.config.yml`.
|
|
21
|
+
This Expo parser deliberately remains home-config-only: it does not consume
|
|
22
|
+
checkout configuration or a `JIRA_SERVER` environment override. Its server must
|
|
23
|
+
be a bare HTTPS origin with no userinfo, query, fragment, or path prefix.
|
|
24
|
+
ASCII DNS names, IPv4, and bracketed IPv6 are supported; default port `443`
|
|
25
|
+
and a root slash are canonicalized away. Whitespace, controls, encoded or
|
|
26
|
+
non-ASCII hosts, trailing dots, and port zero are rejected before the token is read.
|
|
22
27
|
- `gh` CLI authenticated
|
|
23
28
|
- Playwright MCP server running (browser tools available)
|
|
24
29
|
- Dev server running
|
|
@@ -25,6 +25,7 @@ Output (JSON):
|
|
|
25
25
|
}
|
|
26
26
|
"""
|
|
27
27
|
|
|
28
|
+
import ipaddress
|
|
28
29
|
import json
|
|
29
30
|
import os
|
|
30
31
|
import re
|
|
@@ -33,6 +34,31 @@ import sys
|
|
|
33
34
|
import urllib.request
|
|
34
35
|
from base64 import b64encode
|
|
35
36
|
from pathlib import Path
|
|
37
|
+
from urllib.parse import urlsplit
|
|
38
|
+
|
|
39
|
+
|
|
40
|
+
def server_origin(server):
|
|
41
|
+
"""Validate the base parser's bare HTTPS grammar before accessing a token."""
|
|
42
|
+
if not server or any(char <= " " or char >= "\x7f" for char in server):
|
|
43
|
+
return ""
|
|
44
|
+
try:
|
|
45
|
+
parsed = urlsplit(server)
|
|
46
|
+
if (parsed.scheme != "https" or "@" in parsed.netloc
|
|
47
|
+
or "?" in server or "#" in server
|
|
48
|
+
or parsed.path not in ("", "/") or not parsed.hostname):
|
|
49
|
+
return ""
|
|
50
|
+
host = parsed.hostname
|
|
51
|
+
if parsed.netloc.startswith("["):
|
|
52
|
+
host = f"[{ipaddress.IPv6Address(host).compressed}]"
|
|
53
|
+
elif not re.fullmatch(r"[a-z0-9]([a-z0-9-]*[a-z0-9])?(\.[a-z0-9]([a-z0-9-]*[a-z0-9])?)*", host):
|
|
54
|
+
return ""
|
|
55
|
+
port = parsed.port
|
|
56
|
+
if port == 0:
|
|
57
|
+
return ""
|
|
58
|
+
suffix = f":{port}" if port is not None and port != 443 else ""
|
|
59
|
+
return f"https://{host}{suffix}"
|
|
60
|
+
except ValueError:
|
|
61
|
+
return ""
|
|
36
62
|
|
|
37
63
|
|
|
38
64
|
def get_jira_config():
|
|
@@ -51,6 +77,12 @@ def get_jira_config():
|
|
|
51
77
|
elif line.startswith("login:"):
|
|
52
78
|
login = line.split(":", 1)[1].strip()
|
|
53
79
|
|
|
80
|
+
# Home configuration remains the operator-owned source; never trust checkout input here.
|
|
81
|
+
server = server_origin(server)
|
|
82
|
+
if not server:
|
|
83
|
+
print("ERROR: Jira server must be a bare HTTPS origin without userinfo, path, query or fragment.", file=sys.stderr)
|
|
84
|
+
sys.exit(1)
|
|
85
|
+
|
|
54
86
|
token = os.environ.get("JIRA_API_TOKEN", "")
|
|
55
87
|
if not token:
|
|
56
88
|
print("ERROR: JIRA_API_TOKEN env var not set", file=sys.stderr)
|
|
@@ -17,8 +17,13 @@ Read a JIRA ticket's Validation Journey, execute it via Playwright MCP browser t
|
|
|
17
17
|
## Prerequisites
|
|
18
18
|
|
|
19
19
|
- `JIRA_API_TOKEN` environment variable set
|
|
20
|
-
- `jira-cli` configured
|
|
21
|
-
|
|
20
|
+
- `jira-cli` configured in the operator's `~/.config/.jira/.config.yml`.
|
|
21
|
+
This Expo parser deliberately remains home-config-only: it does not consume
|
|
22
|
+
checkout configuration or a `JIRA_SERVER` environment override. Its server must
|
|
23
|
+
be a bare HTTPS origin with no userinfo, query, fragment, or path prefix.
|
|
24
|
+
ASCII DNS names, IPv4, and bracketed IPv6 are supported; default port `443`
|
|
25
|
+
and a root slash are canonicalized away. Whitespace, controls, encoded or
|
|
26
|
+
non-ASCII hosts, trailing dots, and port zero are rejected before the token is read.
|
|
22
27
|
- `gh` CLI authenticated
|
|
23
28
|
- Playwright MCP server running (browser tools available)
|
|
24
29
|
- Dev server running
|
|
@@ -25,6 +25,7 @@ Output (JSON):
|
|
|
25
25
|
}
|
|
26
26
|
"""
|
|
27
27
|
|
|
28
|
+
import ipaddress
|
|
28
29
|
import json
|
|
29
30
|
import os
|
|
30
31
|
import re
|
|
@@ -33,6 +34,31 @@ import sys
|
|
|
33
34
|
import urllib.request
|
|
34
35
|
from base64 import b64encode
|
|
35
36
|
from pathlib import Path
|
|
37
|
+
from urllib.parse import urlsplit
|
|
38
|
+
|
|
39
|
+
|
|
40
|
+
def server_origin(server):
|
|
41
|
+
"""Validate the base parser's bare HTTPS grammar before accessing a token."""
|
|
42
|
+
if not server or any(char <= " " or char >= "\x7f" for char in server):
|
|
43
|
+
return ""
|
|
44
|
+
try:
|
|
45
|
+
parsed = urlsplit(server)
|
|
46
|
+
if (parsed.scheme != "https" or "@" in parsed.netloc
|
|
47
|
+
or "?" in server or "#" in server
|
|
48
|
+
or parsed.path not in ("", "/") or not parsed.hostname):
|
|
49
|
+
return ""
|
|
50
|
+
host = parsed.hostname
|
|
51
|
+
if parsed.netloc.startswith("["):
|
|
52
|
+
host = f"[{ipaddress.IPv6Address(host).compressed}]"
|
|
53
|
+
elif not re.fullmatch(r"[a-z0-9]([a-z0-9-]*[a-z0-9])?(\.[a-z0-9]([a-z0-9-]*[a-z0-9])?)*", host):
|
|
54
|
+
return ""
|
|
55
|
+
port = parsed.port
|
|
56
|
+
if port == 0:
|
|
57
|
+
return ""
|
|
58
|
+
suffix = f":{port}" if port is not None and port != 443 else ""
|
|
59
|
+
return f"https://{host}{suffix}"
|
|
60
|
+
except ValueError:
|
|
61
|
+
return ""
|
|
36
62
|
|
|
37
63
|
|
|
38
64
|
def get_jira_config():
|
|
@@ -51,6 +77,12 @@ def get_jira_config():
|
|
|
51
77
|
elif line.startswith("login:"):
|
|
52
78
|
login = line.split(":", 1)[1].strip()
|
|
53
79
|
|
|
80
|
+
# Home configuration remains the operator-owned source; never trust checkout input here.
|
|
81
|
+
server = server_origin(server)
|
|
82
|
+
if not server:
|
|
83
|
+
print("ERROR: Jira server must be a bare HTTPS origin without userinfo, path, query or fragment.", file=sys.stderr)
|
|
84
|
+
sys.exit(1)
|
|
85
|
+
|
|
54
86
|
token = os.environ.get("JIRA_API_TOKEN", "")
|
|
55
87
|
if not token:
|
|
56
88
|
print("ERROR: JIRA_API_TOKEN env var not set", file=sys.stderr)
|
|
@@ -17,8 +17,13 @@ Read a JIRA ticket's Validation Journey, execute it via Playwright MCP browser t
|
|
|
17
17
|
## Prerequisites
|
|
18
18
|
|
|
19
19
|
- `JIRA_API_TOKEN` environment variable set
|
|
20
|
-
- `jira-cli` configured
|
|
21
|
-
|
|
20
|
+
- `jira-cli` configured in the operator's `~/.config/.jira/.config.yml`.
|
|
21
|
+
This Expo parser deliberately remains home-config-only: it does not consume
|
|
22
|
+
checkout configuration or a `JIRA_SERVER` environment override. Its server must
|
|
23
|
+
be a bare HTTPS origin with no userinfo, query, fragment, or path prefix.
|
|
24
|
+
ASCII DNS names, IPv4, and bracketed IPv6 are supported; default port `443`
|
|
25
|
+
and a root slash are canonicalized away. Whitespace, controls, encoded or
|
|
26
|
+
non-ASCII hosts, trailing dots, and port zero are rejected before the token is read.
|
|
22
27
|
- `gh` CLI authenticated
|
|
23
28
|
- Playwright MCP server running (browser tools available)
|
|
24
29
|
- Dev server running
|
|
@@ -25,6 +25,7 @@ Output (JSON):
|
|
|
25
25
|
}
|
|
26
26
|
"""
|
|
27
27
|
|
|
28
|
+
import ipaddress
|
|
28
29
|
import json
|
|
29
30
|
import os
|
|
30
31
|
import re
|
|
@@ -33,6 +34,31 @@ import sys
|
|
|
33
34
|
import urllib.request
|
|
34
35
|
from base64 import b64encode
|
|
35
36
|
from pathlib import Path
|
|
37
|
+
from urllib.parse import urlsplit
|
|
38
|
+
|
|
39
|
+
|
|
40
|
+
def server_origin(server):
|
|
41
|
+
"""Validate the base parser's bare HTTPS grammar before accessing a token."""
|
|
42
|
+
if not server or any(char <= " " or char >= "\x7f" for char in server):
|
|
43
|
+
return ""
|
|
44
|
+
try:
|
|
45
|
+
parsed = urlsplit(server)
|
|
46
|
+
if (parsed.scheme != "https" or "@" in parsed.netloc
|
|
47
|
+
or "?" in server or "#" in server
|
|
48
|
+
or parsed.path not in ("", "/") or not parsed.hostname):
|
|
49
|
+
return ""
|
|
50
|
+
host = parsed.hostname
|
|
51
|
+
if parsed.netloc.startswith("["):
|
|
52
|
+
host = f"[{ipaddress.IPv6Address(host).compressed}]"
|
|
53
|
+
elif not re.fullmatch(r"[a-z0-9]([a-z0-9-]*[a-z0-9])?(\.[a-z0-9]([a-z0-9-]*[a-z0-9])?)*", host):
|
|
54
|
+
return ""
|
|
55
|
+
port = parsed.port
|
|
56
|
+
if port == 0:
|
|
57
|
+
return ""
|
|
58
|
+
suffix = f":{port}" if port is not None and port != 443 else ""
|
|
59
|
+
return f"https://{host}{suffix}"
|
|
60
|
+
except ValueError:
|
|
61
|
+
return ""
|
|
36
62
|
|
|
37
63
|
|
|
38
64
|
def get_jira_config():
|
|
@@ -51,6 +77,12 @@ def get_jira_config():
|
|
|
51
77
|
elif line.startswith("login:"):
|
|
52
78
|
login = line.split(":", 1)[1].strip()
|
|
53
79
|
|
|
80
|
+
# Home configuration remains the operator-owned source; never trust checkout input here.
|
|
81
|
+
server = server_origin(server)
|
|
82
|
+
if not server:
|
|
83
|
+
print("ERROR: Jira server must be a bare HTTPS origin without userinfo, path, query or fragment.", file=sys.stderr)
|
|
84
|
+
sys.exit(1)
|
|
85
|
+
|
|
54
86
|
token = os.environ.get("JIRA_API_TOKEN", "")
|
|
55
87
|
if not token:
|
|
56
88
|
print("ERROR: JIRA_API_TOKEN env var not set", file=sys.stderr)
|
|
@@ -17,8 +17,13 @@ Read a JIRA ticket's Validation Journey, execute it via Playwright MCP browser t
|
|
|
17
17
|
## Prerequisites
|
|
18
18
|
|
|
19
19
|
- `JIRA_API_TOKEN` environment variable set
|
|
20
|
-
- `jira-cli` configured
|
|
21
|
-
|
|
20
|
+
- `jira-cli` configured in the operator's `~/.config/.jira/.config.yml`.
|
|
21
|
+
This Expo parser deliberately remains home-config-only: it does not consume
|
|
22
|
+
checkout configuration or a `JIRA_SERVER` environment override. Its server must
|
|
23
|
+
be a bare HTTPS origin with no userinfo, query, fragment, or path prefix.
|
|
24
|
+
ASCII DNS names, IPv4, and bracketed IPv6 are supported; default port `443`
|
|
25
|
+
and a root slash are canonicalized away. Whitespace, controls, encoded or
|
|
26
|
+
non-ASCII hosts, trailing dots, and port zero are rejected before the token is read.
|
|
22
27
|
- `gh` CLI authenticated
|
|
23
28
|
- Playwright MCP server running (browser tools available)
|
|
24
29
|
- Dev server running
|
|
@@ -25,6 +25,7 @@ Output (JSON):
|
|
|
25
25
|
}
|
|
26
26
|
"""
|
|
27
27
|
|
|
28
|
+
import ipaddress
|
|
28
29
|
import json
|
|
29
30
|
import os
|
|
30
31
|
import re
|
|
@@ -33,6 +34,31 @@ import sys
|
|
|
33
34
|
import urllib.request
|
|
34
35
|
from base64 import b64encode
|
|
35
36
|
from pathlib import Path
|
|
37
|
+
from urllib.parse import urlsplit
|
|
38
|
+
|
|
39
|
+
|
|
40
|
+
def server_origin(server):
|
|
41
|
+
"""Validate the base parser's bare HTTPS grammar before accessing a token."""
|
|
42
|
+
if not server or any(char <= " " or char >= "\x7f" for char in server):
|
|
43
|
+
return ""
|
|
44
|
+
try:
|
|
45
|
+
parsed = urlsplit(server)
|
|
46
|
+
if (parsed.scheme != "https" or "@" in parsed.netloc
|
|
47
|
+
or "?" in server or "#" in server
|
|
48
|
+
or parsed.path not in ("", "/") or not parsed.hostname):
|
|
49
|
+
return ""
|
|
50
|
+
host = parsed.hostname
|
|
51
|
+
if parsed.netloc.startswith("["):
|
|
52
|
+
host = f"[{ipaddress.IPv6Address(host).compressed}]"
|
|
53
|
+
elif not re.fullmatch(r"[a-z0-9]([a-z0-9-]*[a-z0-9])?(\.[a-z0-9]([a-z0-9-]*[a-z0-9])?)*", host):
|
|
54
|
+
return ""
|
|
55
|
+
port = parsed.port
|
|
56
|
+
if port == 0:
|
|
57
|
+
return ""
|
|
58
|
+
suffix = f":{port}" if port is not None and port != 443 else ""
|
|
59
|
+
return f"https://{host}{suffix}"
|
|
60
|
+
except ValueError:
|
|
61
|
+
return ""
|
|
36
62
|
|
|
37
63
|
|
|
38
64
|
def get_jira_config():
|
|
@@ -51,6 +77,12 @@ def get_jira_config():
|
|
|
51
77
|
elif line.startswith("login:"):
|
|
52
78
|
login = line.split(":", 1)[1].strip()
|
|
53
79
|
|
|
80
|
+
# Home configuration remains the operator-owned source; never trust checkout input here.
|
|
81
|
+
server = server_origin(server)
|
|
82
|
+
if not server:
|
|
83
|
+
print("ERROR: Jira server must be a bare HTTPS origin without userinfo, path, query or fragment.", file=sys.stderr)
|
|
84
|
+
sys.exit(1)
|
|
85
|
+
|
|
54
86
|
token = os.environ.get("JIRA_API_TOKEN", "")
|
|
55
87
|
if not token:
|
|
56
88
|
print("ERROR: JIRA_API_TOKEN env var not set", file=sys.stderr)
|