Remove --url flag from page observe command

[giordano-lucas]

page observe should only observe the current page state of an active session, not navigate to a new URL.

Greptile Overview

Greptile Summary

This PR removes the --url flag from the page observe command, aligning the behavior to only observe the current page state rather than navigating to a new URL. The code changes in internal/cmd/ are clean and complete - the variable, flag registration, and usage logic were all properly removed, along with associated unit tests.

However, the PR has critical issues:

• Integration tests broken: 9 integration test files in tests/integration/ still use the removed --url flag, which will cause runtime failures
• Documentation outdated: 2 documentation files in skills/notte-browser/ still reference the --url flag that no longer exists

These need to be fixed before merging to prevent broken tests and user confusion.

Confidence Score: 1/5

• This PR cannot be merged safely - it will break 9 integration tests
• The implementation changes are clean, but critical references to the removed flag remain in integration tests (9 instances) and documentation (2 instances). These will cause test failures and user confusion.
• Pay close attention to tests/integration/page_test.go and tests/integration/sessions_test.go - all uses of the --url flag must be removed

Important Files Changed

Filename	Overview
tests/integration/page_test.go	6 integration tests still use the removed --url flag - will cause runtime failures
tests/integration/sessions_test.go	3 integration tests still use the removed --url flag - will cause runtime failures
skills/notte-browser/SKILL.md	Documentation still shows [--url <url>] option for page observe command
skills/notte-browser/references/session-management.md	Documentation includes example using URL argument with page observe that no longer works

_{Last reviewed commit: 221c1b2}

[greptile-apps]

_{4 files reviewed, 11 comments}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

Additional Comments (11)

skills/notte-browser/SKILL.md
Remove [--url <url>] from the documentation since the flag no longer exists

notte page observe

---

skills/notte-browser/references/session-management.md
Remove the "Navigate and observe" example since page observe no longer accepts a URL argument

# Observe current page
notte page observe

---

tests/integration/sessions_test.go
Integration test still uses the removed --url flag - this will fail at runtime. Remove the --url flag and its value from this command.

	result = runCLIWithTimeout(t, 120*time.Second, "sessions", "observe", "--session-id", sessionID)

---

tests/integration/sessions_test.go
Remove --url flag

	result = runCLIWithTimeout(t, 120*time.Second, "sessions", "observe", "--session-id", sessionID)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

---

tests/integration/sessions_test.go
Remove --url flag

	result = runCLIWithTimeout(t, 120*time.Second, "sessions", "observe", "--session-id", sessionID)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

---

tests/integration/page_test.go
Remove --url flag

	result = runCLIWithTimeout(t, 120*time.Second, "sessions", "observe", "--session-id", sessionID)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

---

tests/integration/page_test.go
Remove --url flag

	result = runCLIWithTimeout(t, 120*time.Second, "sessions", "observe", "--session-id", sessionID)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

---

tests/integration/page_test.go
Remove --url flag

	result = runCLIWithTimeout(t, 120*time.Second, "sessions", "observe", "--session-id", sessionID)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

---

tests/integration/page_test.go
Remove --url flag

	result = runCLIWithTimeout(t, 120*time.Second, "sessions", "observe", "--session-id", sessionID)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

---

tests/integration/page_test.go
Remove --url flag

	result = runCLIWithTimeout(t, 120*time.Second, "sessions", "observe", "--session-id", sessionID)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

---

tests/integration/page_test.go
Remove --url flag

	result = runCLIWithTimeout(t, 120*time.Second, "sessions", "observe", "--session-id", sessionID)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}