scripting: fix LND RPC authentication using lndclient wrappers

This commit fixes LND RPC authentication by switching from raw gRPC
clients to lndclient wrappers that provide proper macaroon authentication.

Changes:
- Update LNDClients to use lndclient.LightningClient and
  lndclient.RouterClient instead of raw gRPC clients
- Add getRawLightningClient() and getRawRouterClient() helpers that use
  RawClientWithMacAuth() for authenticated raw client access
- Update all 21 LND builtin methods to use the new pattern
- Add InMemoryStore implementation for testing
- Wire up LND clients in terminal.go after LND connects
- Fix integration test function signatures

All 7 scripts integration tests now pass:
- scripts_basic_crud
- scripts_validation
- scripts_execution
- scripts_kv_store
- scripts_lnd_access
- scripts_builtins
- scripts_kv_builtins

Author: sputn1ck

itest/litd_scripts_test.go

@@ -15,8 +15,9 @@ import (
 )
 
 // testScriptBasicCRUD tests basic script create, read, update, delete operations.
-func testScriptBasicCRUD(t *harnessTest, net *NetworkHarness) {
-	ctx := context.Background()
+func testScriptBasicCRUD(ctx context.Context, net *NetworkHarness,
+	t *harnessTest) {
+
 	ctxt, cancel := context.WithTimeout(ctx, defaultTimeout)
 	defer cancel()
 
@@ -98,8 +99,9 @@ def main():
 }
 
 // testScriptValidation tests script syntax validation.
-func testScriptValidation(t *harnessTest, net *NetworkHarness) {
-	ctx := context.Background()
+func testScriptValidation(ctx context.Context, net *NetworkHarness,
+	t *harnessTest) {
+
 	ctxt, cancel := context.WithTimeout(ctx, defaultTimeout)
 	defer cancel()
 
@@ -148,8 +150,9 @@ def helper():
 }
 
 // testScriptExecution tests basic script execution.
-func testScriptExecution(t *harnessTest, net *NetworkHarness) {
-	ctx := context.Background()
+func testScriptExecution(ctx context.Context, net *NetworkHarness,
+	t *harnessTest) {
+
 	ctxt, cancel := context.WithTimeout(ctx, defaultTimeout)
 	defer cancel()
 
@@ -220,8 +223,9 @@ def main(x=10, y=20):
 }
 
 // testScriptKVStore tests the script KV store operations.
-func testScriptKVStore(t *harnessTest, net *NetworkHarness) {
-	ctx := context.Background()
+func testScriptKVStore(ctx context.Context, net *NetworkHarness,
+	t *harnessTest) {
+
 	ctxt, cancel := context.WithTimeout(ctx, defaultTimeout)
 	defer cancel()
 
@@ -298,8 +302,9 @@ func testScriptKVStore(t *harnessTest, net *NetworkHarness) {
 }
 
 // testScriptWithLNDAccess tests scripts that access LND RPCs.
-func testScriptWithLNDAccess(t *harnessTest, net *NetworkHarness) {
-	ctx := context.Background()
+func testScriptWithLNDAccess(ctx context.Context, net *NetworkHarness,
+	t *harnessTest) {
+
 	ctxt, cancel := context.WithTimeout(ctx, defaultTimeout)
 	defer cancel()
 
@@ -350,8 +355,9 @@ def main():
 }
 
 // testScriptBuiltins tests the built-in functions.
-func testScriptBuiltins(t *harnessTest, net *NetworkHarness) {
-	ctx := context.Background()
+func testScriptBuiltins(ctx context.Context, net *NetworkHarness,
+	t *harnessTest) {
+
 	ctxt, cancel := context.WithTimeout(ctx, defaultTimeout)
 	defer cancel()
 
@@ -411,8 +417,9 @@ def main():
 }
 
 // testScriptWithKVBuiltins tests scripts using built-in KV functions.
-func testScriptWithKVBuiltins(t *harnessTest, net *NetworkHarness) {
-	ctx := context.Background()
+func testScriptWithKVBuiltins(ctx context.Context, net *NetworkHarness,
+	t *harnessTest) {
+
 	ctxt, cancel := context.WithTimeout(ctx, defaultTimeout)
 	defer cancel()