fix: Object.Filter() method now properly includes attributes when requested (#388)

Resolves #387: The Object.Filter() method was returning empty results when
attributes=true because it wasn't including the includeAttributes=true parameter
in the API request URL.

Changes:
- Fixed logic in Filter() method to explicitly set includeAttributes parameter
  for both true and false cases
- Added comprehensive test coverage for attributes=true scenario
- Maintains backward compatibility with existing behavior

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude <noreply@anthropic.com>

Author: fank

assets/internal/object_impl.go

@@ -159,7 +159,9 @@ func (i *internalObjectImpl) Filter(ctx context.Context, workspaceID, aql string
 	params.Add("startAt", strconv.Itoa(startAt))
 	params.Add("maxResults", strconv.Itoa(maxResults))
 
-	if !attributes {
+	if attributes {
+		params.Add("includeAttributes", "true")
+	} else {
 		params.Add("includeAttributes", "false")
 	}
 

assets/internal/object_impl_test.go

@@ -1184,6 +1184,32 @@ func Test_internalObjectImpl_Filter(t *testing.T) {
 			wantErr: true,
 			Err:     model.ErrNoAqlQuery,
 		},
+		{
+			name: "when the parameters are correct with attributes enabled",
+			args: args{
+				ctx:         context.Background(),
+				workspaceID: "workspace-uuid-sample",
+				aql:         "objectType = Office AND Name LIKE SYD",
+				attributes:  true,
+				startAt:     50,
+				maxResults:  50,
+			},
+			on: func(fields *fields) {
+				client := mocks.NewConnector(t)
+				client.On("NewRequest",
+					context.Background(),
+					http.MethodPost,
+					"jsm/assets/workspace/workspace-uuid-sample/v1/object/aql?includeAttributes=true&maxResults=50&startAt=50",
+					"",
+					map[string]interface{}{"qlQuery": "objectType = Office AND Name LIKE SYD"}).
+					Return(&http.Request{}, nil)
+				client.On("Call",
+					&http.Request{},
+					&model.ObjectListResultScheme{}).
+					Return(&model.ResponseScheme{}, nil)
+				fields.c = client
+			},
+		},
 	}
 
 	for _, testCase := range testCases {