Skip to content

fix(e2e): bind fake APIs to the OpenShell bridge #10757

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
prekshivyas merged 32 commits into from
Sep 1, 2026
Merged

fix(e2e): bind fake APIs to the OpenShell bridge #10757

Show file tree
Hide file tree
Changes from 12 commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
e87e0b3
fix(e2e): restore sandbox fake API reachability
prekshivyas Sep 1, 2026
7165c49
Merge remote-tracking branch 'origin/main' into codex/fix-e2e-proxy-b…
prekshivyas Sep 1, 2026
e450ab9
test(e2e): map Hermes Discord bridge proof
prekshivyas Sep 1, 2026
55d311b
fix(e2e): harden fake API proxy readiness
prekshivyas Sep 1, 2026
ba045b8
test(e2e): complete fake API proxy migration
prekshivyas Sep 1, 2026
5591017
test(e2e): capture fake API startup diagnostics
prekshivyas Sep 1, 2026
6333239
test(e2e): prove proxy upstream reachability
prekshivyas Sep 1, 2026
cef370f
test(e2e): preserve proxy failure diagnostics
prekshivyas Sep 1, 2026
1406187
fix(e2e): handle proxy readiness socket errors
cv Sep 1, 2026
d33e618
test(e2e): handle proxy readiness resets
prekshivyas Sep 1, 2026
d10fdcf
fix(e2e): tighten Discord fixture diagnostics
cv Sep 1, 2026
d1f1bcb
refactor(e2e): consolidate fake API readiness
prekshivyas Sep 1, 2026
848f5fd
test(e2e): map pairing helper parity proof
cv Sep 1, 2026
d545462
test(e2e): remove retired Vitest import
cv Sep 1, 2026
b0bc674
test(e2e): own pairing helper parity
prekshivyas Sep 1, 2026
639f64b
test(e2e): clarify best-effort diagnostics
cv Sep 1, 2026
be75a56
test(e2e): verify fake API Docker topology
prekshivyas Sep 1, 2026
2b796e5
test(e2e): strengthen fake API topology proofs
cv Sep 1, 2026
404dba6
test(e2e): use valid Discord fixture contract
cv Sep 1, 2026
b098f29
test(e2e): complete topology regression proof
prekshivyas Sep 1, 2026
a360c93
refactor(e2e): inline Hermes fixture setup
cv Sep 1, 2026
8f1fccc
test(e2e): use independent topology evidence
prekshivyas Sep 1, 2026
2cba16d
fix(e2e): preserve stopped fixture diagnostics
prekshivyas Sep 1, 2026
be5d0c6
test(e2e): prove fake API Docker topology
prekshivyas Sep 1, 2026
d672507
test(e2e): register topology proof parity
prekshivyas Sep 1, 2026
7e89676
fix(e2e): scope Hermes Discord policy binary
prekshivyas Sep 1, 2026
9eff794
test(e2e): assert diagnostic artifact outcomes
prekshivyas Sep 1, 2026
dc16294
Merge remote-tracking branch 'origin/main' into codex/fix-e2e-proxy-b…
prekshivyas Sep 1, 2026
18ee35f
fix(e2e): keep proxy environment credential-free
prekshivyas Sep 1, 2026
3c8d563
refactor(e2e): consolidate topology validation
prekshivyas Sep 1, 2026
3796452
test(e2e): derive topology from Docker commands
prekshivyas Sep 1, 2026
b3bb16e
test(e2e): isolate proxy proof environment
prekshivyas Sep 1, 2026
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
501 changes: 0 additions & 501 deletions test/e2e/lib/discord-rest-policy-proof.sh
View file
Open in desktop

This file was deleted.

157 changes: 0 additions & 157 deletions test/e2e/lib/fake-discord-message-api.cjs
View file
Open in desktop

This file was deleted.

75 changes: 0 additions & 75 deletions test/e2e/lib/fake-discord-rest-api.cjs
View file
Open in desktop

This file was deleted.

9 changes: 2 additions & 7 deletions test/e2e/live/hermes-discord.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ const DISCORD_SERVER_IDS = process.env.DISCORD_SERVER_IDS ?? "149159099275359059
const DISCORD_ALLOWED_IDS = process.env.DISCORD_ALLOWED_IDS ?? "1005536447329222676";
const DISCORD_REQUIRE_MENTION = process.env.DISCORD_REQUIRE_MENTION ?? "0";
const HERMES_HEALTH_URL = "http://localhost:8642/health";
const FAKE_DISCORD_HOST = "host.docker.internal";
const FAKE_DISCORD_HOST = "host.openshell.internal";
const HERMES_DISCORD_HTTP_PROXY_GATEWAY_TEMPLATE = hermesDiscordHttpProxyWebSocketUrl(
"{host}",
"{port}",
Expand Down Expand Up @@ -125,7 +125,6 @@ async function startHermesFakeDiscordGateway(
imageScript: "fake-discord-gateway.cjs",
containerPrefix: "nemoclaw-fake-discord-hermes",
portEnv: "FAKE_DISCORD_GATEWAY_PORT",
portFileEnv: "FAKE_DISCORD_GATEWAY_PORT_FILE",
captureFileEnv: "FAKE_DISCORD_GATEWAY_CAPTURE_FILE",
expectedEnv: { FAKE_DISCORD_GATEWAY_EXPECTED_TOKEN: token },
env,
Expand Down Expand Up @@ -153,10 +152,6 @@ async function applyHermesFakeDiscordPolicy(options: {
"--add-allow",
`${FAKE_DISCORD_HOST}:${options.api.port}:WEBSOCKET_TEXT:/**`,
"--binary",
"/usr/local/bin/node",
"--binary",
"/usr/bin/node",
"--binary",
"/usr/local/bin/python3",
"--binary",
"/usr/bin/python3",
Expand Down Expand Up @@ -267,7 +262,7 @@ results = []

async def main():
port = int(os.environ["FAKE_DISCORD_GATEWAY_CLIENT_PORT"])
host = os.environ.get("FAKE_DISCORD_GATEWAY_CLIENT_HOST", "host.docker.internal")
host = os.environ.get("FAKE_DISCORD_GATEWAY_CLIENT_HOST", "host.openshell.internal")
token = read_env_token()
client = discord.Client(intents=discord.Intents.none())
setup = getattr(client, "_async_setup_hook", None)
Expand Down
Loading
Loading